plone.restapi icon indicating copy to clipboard operation
plone.restapi copied to clipboard

allow anonymous users to access @types endpoint

Open erral opened this issue 3 years ago • 5 comments

Addresses #1409

erral avatar May 09 '22 14:05 erral

@erral thanks for creating this Pull Request and helping to improve Plone!

TL;DR: Finish pushing changes, pass all other checks, then paste a comment:

@jenkins-plone-org please run jobs

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!

mister-roboto avatar May 09 '22 14:05 mister-roboto

Deploy Preview for plone-restapi canceled.

Name Link
Latest commit 12197d7f0d312e8775fef08b8ada4600fd6cd2c9
Latest deploy log https://app.netlify.com/sites/plone-restapi/deploys/64e84ead2fa12d0008a8e252

netlify[bot] avatar May 09 '22 14:05 netlify[bot]

@erral what's the status of this PR? Can you elaborate your use case?

tisto avatar Aug 25 '23 06:08 tisto

I don't recall why we added this PR...

I think it must be somehow related to the work we did during the Beethoven Sprint when we added in Volto a default view for schema based content-types

erral avatar Aug 25 '23 06:08 erral

@tisto @erral I'm aware of 2 use cases that came up for this.

The first is the default schema-based view, like you said. That still does not work for anonymous users. But, this change stalled because there were some concerns about exposing private values in the schemas (such as vocabulary values).

The second is the use case of showing content type titles in listings. We solved that a different way in https://github.com/plone/plone.restapi/pull/1355

davisagli avatar Aug 25 '23 15:08 davisagli