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

Userschema for registration vs. userschema for user profile #1873

Closed
erral opened this issue Jan 30, 2025 · 4 comments · Fixed by #1874
Closed

Userschema for registration vs. userschema for user profile #1873

erral opened this issue Jan 30, 2025 · 4 comments · Fixed by #1874
Assignees

Comments

@erral
Copy link
Member

erral commented Jan 30, 2025

Our current @userschema endpoint returns the full user schema as defined in Plone.

But in Plone we have different user schemas, one for the registration form and the other one for the user profile form.

They are, in essence, the same user profile, but filtered according to the field settings, where one can set whether a given field will be shown in the registration form, in the user profile form, in both, or in neither of them.

So I propose that we add a parameter to the @userschema endpoint, so that only user registration fields or only user profile fields can be requested.

@ionlizarazu

@erral erral self-assigned this Jan 30, 2025
@erral
Copy link
Member Author

erral commented Jan 30, 2025

This can be relevant for plone/volto#6435

@erral
Copy link
Member Author

erral commented Jan 30, 2025

Or we can create a different endpoint @registration-userschema that returns the fiels available for registration.

@davisagli
Copy link
Member

@erral It makes sense that we need a separate way to get the userschema for registration. Can we make the endpoint @userschema/registration instead of @registration-userschema? It just keeps it organized a bit better, I think, and we can document both userschema endpoints together.

@erral
Copy link
Member Author

erral commented Jan 31, 2025

Yep, nice idea

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

Successfully merging a pull request may close this issue.

2 participants