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

Connect admin-pages under the same view #1232

Merged
merged 1 commit into from Oct 6, 2023
Merged

Conversation

norbye
Copy link
Member

@norbye norbye commented Sep 10, 2023

Both group admins and admission admins (group leader+recruiting and abakus/revue leader) now use the same admin pages to reduce duplicate code.

The editing of questions for groups has been moved to separate URLs, that only group leader+recruiting has access to.

This fixes the issue we have right now where the leaders of the different groups in the revue do not have access to edit their own texts, because they also have permission to view all applications.

Minor changes

  • Improved typing of userdata fetched from Django

Preview

Application list
image

Edit group details
image

No permission to edit group details
image

Resolves ABA-411

@norbye norbye added enhancement Pull requests that make or issues that suggest enhancements, instead of just purely new features review-needed Pull requests that need review technical-debt Pull requests that reduces technical debt labels Sep 10, 2023
@norbye norbye requested a review from a team September 10, 2023 08:17
@linear
Copy link

linear bot commented Sep 10, 2023

ABA-411 Connect admin-pages

Currently, the adminpage where you see the results of the admission and edit the questions are more "connected" to actually sending your own application (through the navbar). It would be nice to instead make it simpler to navigate between the two admin-views (if allowed).

Could possibly be nice to also allow navigating to the admission, but that's not as important imo..

@norbye norbye merged commit d53d37f into master Oct 6, 2023
3 checks passed
@norbye norbye deleted the connect-admin-pages branch October 6, 2023 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Pull requests that make or issues that suggest enhancements, instead of just purely new features review-needed Pull requests that need review technical-debt Pull requests that reduces technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant