Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vocabulary entries returned do not include their token #106

Open
jackahl opened this issue May 4, 2022 · 0 comments
Open

vocabulary entries returned do not include their token #106

jackahl opened this issue May 4, 2022 · 0 comments

Comments

@jackahl
Copy link
Member

jackahl commented May 4, 2022

When a vocabulary is used by a querystring param only value and title are returned. This also exposes the values via the REST api which can become problematic, as values are not necessarily string values, but can potentially even be python objects.

Therefore we should aim to rather use the token instead of value when used in Volto via REST api. To prepare for that change, the token should be included in the api response. In a later stage we should refactor it so, that the api response does not include the value at all. But that should be done in plone.restapi and volto respectively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant