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

Eregcsc 2290 edit button uploaded items #1051

Closed
wants to merge 22 commits into from

Conversation

peggles2
Copy link
Contributor

@peggles2 peggles2 commented Nov 15, 2023

Resolves #EREGCSC-2290.

Description-
Uploaded files related to a subject should dislay an edit button if the user has appropriate jobcodes (Eregs-admin or Eregs-manager)

This pull request changes...

Backend:

Frontend:

Cypress Tests:
solution/ui/e2e/cypress/e2e/policy-repository.spec.cy.js
tests the ui to see the edit button if user is an admin or manager.

Functional Tests:

Steps to manually verify this change...
None admin:

  1. Sign in through the admin panel.
    Screen Shot 2023-11-13 at 2 18 26 PM
  2. click on users->click on the user name
  3. scroll down to Groups and add yourself to eregs-reader
    Screen Shot 2023-11-13 at 2 19 49 PM
  4. visit policy-repository?q=dental. notice that the listed files to the right do not have an edit option.
    Screen Shot 2023-11-13 at 2 55 32 PM

ADMIN:

  1. Sign in through the admin panel.
    Screen Shot 2023-11-13 at 2 18 26 PM

  2. click on users->click on the user name

  3. scroll down to Groups and add yourself to eregs-reader
    Screen Shot 2023-11-13 at 2 19 49 PM

  4. visit policy-repository?q=dental. notice that the listed files to the right have an edit option.
    Screen Shot 2023-11-13 at 2 23 56 PM

Copy link

✨ See the Django Site in action

Copy link

✨ See the Django Site in action

Copy link

✨ See the Django Site in action

Copy link

✨ See the Django Site in action

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