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

feat: Added project admin UI for user edit page [PT-186125988] #1097

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

dougmartin
Copy link
Member

This adds a list of checkboxes, one per project, on the user edit page that can be checked/unchecked to set the projects the user is the admin of.

This also updates the user and project models to have a second project admin has_many association that only holds the user or project. This allows the standard Rails update_attributes method to update the association based on the checkboxes checked in the form.

Finally, this also updates the styling of the user form, for both the edit and the (unused) new form.

This adds a list of checkboxes, one per project, on the user edit page that can be checked/unchecked to set the projects the user is the admin of.

This also updates the user and project models to have a second project admin has_many association that only holds the user or project.  This allows the standard Rails update_attributes method to update the association based on the checkboxes checked in the form.

Finally, this also updates the styling of the user form, for both the edit and the (unused) new form.
Base automatically changed from 186125987-create-project-admin-model to master November 21, 2023 12:52
@dougmartin dougmartin merged commit d95313d into master Nov 21, 2023
3 of 4 checks passed
@dougmartin dougmartin deleted the 186125988-add-user-project-admin-ui branch November 21, 2023 12:53
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.

1 participant