Conversation
karawoo
left a comment
There was a problem hiding this comment.
Can we add a news bullet please?
Of course, I'll send one in a sec The integration tests look unrelated but I don't know what's up with those: https://github.com/posit-dev/connectapi/actions/runs/22406879377/job/64869098248#step:6:530 |
|
They're failing on main too, it's due to a new field in the API responses. I'll fix separately, you can ignore. |
|
Added news and a version guard |
|
@jonkeane can you update the unit tests so they have the right server version? |
|
@jonkeane actually if you don't mind, i'll just do it -- it'll be a quick fix and will unblock me moving forward with the cran release |
|
the failing test is fixed on main, everything else looks good. I'm going to merge this in order to proceed with the release. |
Intent
Now that connect accepts GUID filters on the hits endpoint, add that to the sdk
Approach
Checklist
NEWS.md(referencing the connected issue if necessary)?devtools::document()?