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

Make date_of_birth optional on the user model #1121

Open
lukasrad02 opened this issue Oct 27, 2023 · 0 comments · May be fixed by #1261
Open

Make date_of_birth optional on the user model #1121

lukasrad02 opened this issue Oct 27, 2023 · 0 comments · May be fixed by #1261
Labels
[C] enhancement Changes to an existing feature making it better [P] minor features or bugs that can wait for some more time

Comments

@lukasrad02
Copy link

Is your feature request related to a problem? Please describe.
If Ephios is using an OIDC IDP that does not provide a claim for the date of birth, this field is not set for all users created trough this IDP. However, since date_of_birth is listed in the required fields, such a user cannot be edited without setting the birthday. When editing a user, the birthday might not be known, so one has to guess it to be able to submit the form.

Describe the solution you'd like
Allow changing a user without specifying a date of birth if it wasn't specified for this user before. The easiest way would be to make the date_of_birth field optional on the UserProfile model.

Describe alternatives you've considered

  • Leave the date_of_birth field required by default, but make it optional when editing users that don't have a date of birth set yet.
  • Leave the date_of_birth field required by default, but make it optional for all user accounts coming from a specific identity provider (configurable) or any identity provider.
  • Allow an administrator to configure whether the date of birth is required or not (for all users).
@jeriox jeriox added [C] enhancement Changes to an existing feature making it better [P] minor features or bugs that can wait for some more time labels Oct 31, 2023
@jeriox jeriox linked a pull request Apr 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] enhancement Changes to an existing feature making it better [P] minor features or bugs that can wait for some more time
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants