Skip to content

UI fixes #743

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

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

UI fixes #743

wants to merge 6 commits into from

Conversation

cecilialau6776
Copy link
Contributor

@cecilialau6776 cecilialau6776 commented Jul 25, 2025

  • Notifications badge on project detail page now only counts the number of notifications the user can see
  • Allocation change request page's approve, deny, and update buttons now have consistent spacing
  • Only users can edit a change request can see the edit button on the list in the allocation detail page
    • I took the condition from test_func() in AllocationChangeDetailView - maybe it's cleaner to have a static function in that class (such as def can_view(user, allocation_obj)) to call
  • Add "Request Change" button to the allocation change requests section of the allocation detail page

Closes #739.

Signed-off-by: Cecilia Lau <cecilialau6776@gmail.com>
Signed-off-by: Cecilia Lau <cecilialau6776@gmail.com>
Signed-off-by: Cecilia Lau <cecilialau6776@gmail.com>
Signed-off-by: Cecilia Lau <cecilialau6776@gmail.com>
Signed-off-by: Cecilia Lau <cecilialau6776@gmail.com>
Signed-off-by: Cecilia Lau <cecilialau6776@gmail.com>
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.

Bug/Feature: Misc UI changes
1 participant