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.

Sign up for GitHub

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

Jump to bottom

Account Permissions: Admins approve request #127

Open
11 tasks
elizabethfoster02 opened this issue Mar 27, 2024 · 0 comments
Open
11 tasks

Account Permissions: Admins approve request #127

elizabethfoster02 opened this issue Mar 27, 2024 · 0 comments

Comments

@elizabethfoster02
Copy link

Purpose
Currently, any account can approve a pending shift request. We want to restrict this so that only admins can approve this request. This means modifying what buttons are available on a shift details page depending on the user's account status (admin or not admin).
So, if a shift is pending, the approve/reject request buttons should only appear if the account is a manager

TODO

  • Claim this ticket by adding your name and moving to "In Progress"
  • Get updated with the codebase
    • git checkout dev
    • git pull
    • THEN create a new branch as per usual
    • npm install (this updates your dependencies)
    • npm run dev
  • Implement description above
  • Create a PR for this ticket
  • Move this ticket into "Ready for Review"

Testing

  • Desktop: show the same shift details page but logged in from an admin account vs an employee account
@elizabethfoster02 elizabethfoster02 converted this from a draft issue Mar 27, 2024
@naomi9501 naomi9501 assigned naomi9501 and S-P-R and unassigned S-P-R and naomi9501 Mar 27, 2024
@S-P-R S-P-R moved this from Todo to Done in JumboCode Casa Myrna Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@S-P-R @elizabethfoster02 @naomi9501