Skip to content
This repository has been archived by the owner on Feb 8, 2023. It is now read-only.

1364 - getUserRole checks for POC roles and applies admin in case of #1365

Merged
merged 2 commits into from
Jun 9, 2020
Merged

1364 - getUserRole checks for POC roles and applies admin in case of #1365

merged 2 commits into from
Jun 9, 2020

Conversation

briandavidson
Copy link
Member

@briandavidson briandavidson commented Jun 9, 2020 •
edited
Loading

Summary

Addresses Issue #1364

This will make it so POC level users will have access to the areas they should.

This pull request is ready to merge when...

  • Feature branch is starts with the issue number
  • Is connected to its original issue via zenhub 👇
  • All tests are passing and meet coverage, linting, and accessibility requirements. And no security vulnerabilities ⚫️(Circle)
  • This code has been reviewed by someone other than the original author

mtlaney
mtlaney approved these changes Jun 9, 2020
View reviewed changes
Copy link
Contributor

@mtlaney mtlaney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved pending lint changes

@briandavidson briandavidson merged commit ea8a457 into dev Jun 9, 2020
@abdul-fs abdul-fs mentioned this pull request Jun 10, 2020
Merged
6 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mtlaney mtlaney mtlaney approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@briandavidson @mtlaney