Conversation
|
@jnptk thanks for creating this Pull Request and helping to improve Plone! TL;DR: Finish pushing changes, pass all other checks, then paste a comment: To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically. Happy hacking! |
|
@davisagli @gforcada @mauritsvanrees it looks like we have a similar problem here in plone.restapi as in plone.api with CI. Can one of you have a look please? |
|
@stevepiercy The problems are with the constraints for Plone 6.2-dev and need to be fixed there. I gave more details in plone/plone.api#591 (comment) You could make the documentation repository use the 9.x.x branch of plone.restapi until this is resolved. |
|
@davisagli when will the constraints get fixed? It doesn't make sense for me to change and revert something that'll be handled within the next week. |
davisagli
left a comment
davisagli
left a comment
There was a problem hiding this comment.
@stevepiercy They were fixed and this passes now.
@jnptk Thanks for fixing the link.
* fix link to querystring operators in docs * news * Rename file extension * Update docs/source/endpoints/querystringsearch.md --------- Co-authored-by: Steve Piercy <web@stevepiercy.com>
4 participants
The link was wrong and led to a 404
📚 Documentation preview 📚: https://plonerestapi--1961.org.readthedocs.build/