David Glick

Results 452 comments of David Glick

@albavilanova No, in order for the REST API to recognize that there is a current user, the user needs to be authenticated in some way, and in order to authenticate...

@tisto Is there some way that we can make the endpoints indicate that a query is needed to get results, in that case? Simply returning an empty list makes it...

It's not immediately obvious to me why the translation isn't working, because plone.restapi tries to do that here: https://github.com/plone/plone.restapi/blob/main/src/plone/restapi/types/adapters.py#L63 I'll transfer the issue to plone.restapi, since it does seem to...

This was _supposed_ to be fixed by https://github.com/plone/plone.restapi/pull/1477. Someone needs to debug and figure out why it's not working.

The code is grouping items by their parent folder, and then it does a copy (or cut) and paste once for each parent folder. I don't think it has a...

`self.controlpanel` isn't None, but it's missing a `title` because of this conditional: https://github.com/plone/plone.restapi/blob/main/src/plone/restapi/controlpanels/__init__.py#L37 Our goal here should be to return a 401 response if the current user doesn't have permission...

@wesleybl UTF-8 is the only charset supported for application/json (see https://stackoverflow.com/a/73074619), so that sounds kind of redundant and non-standard. If you can point me to more information about how leaving...

@wesleybl Maybe, but there can also be some friction from changing it...i.e. tests for the response Content-Type which start to fail, even though there's no real problem. I'm okay with...

@wesleybl Yes, unless you specifically ask for more. That's intentional. (But setting items_total to the limit, instead of the actual total, is not intentional.)