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

Disable access role inputs if there is a value set which is not in list of given values #6

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

eluhr
Copy link
Contributor

@eluhr eluhr commented Mar 11, 2025

The aim of this is to solve the specific problem that if a user does not have the role assigned in the input field, the value of the attribute behind this field is set to NULL.

Q A
Is bugfix? Yes
New feature? No
Breaks BC? No
Tests pass? Where there are no tests, they cannot pass

@eluhr eluhr marked this pull request as draft March 11, 2025 08:42
@eluhr eluhr requested review from schmunk42 and handcode March 11, 2025 11:58
@eluhr eluhr changed the title WIP: Disable access role inputs if there is a value set which is not in list of given values Disable access role inputs if there is a value set which is not in list of given values Mar 11, 2025
@eluhr eluhr marked this pull request as ready for review March 11, 2025 12:53
Copy link
Member

@schmunk42 schmunk42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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 this pull request may close these issues.

2 participants