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

feat(app): Developer role added that can see applications on all supervisor proposals #21

Merged
merged 3 commits into from
Dec 18, 2023

Conversation

mxmlnwbr
Copy link
Collaborator

Description:
Added a new "Developer" role granting access to view applications across all supervisor proposals within the app.

Changes:
Implemented the "Developer" role functionality.
Enabled viewing permissions for applications across supervisor proposals.

Impact:
Users assigned as "Developer" can now access and review applications across the platform (for testing).

@mxmlnwbr mxmlnwbr requested a review from rschlaefli December 15, 2023 10:00
@mxmlnwbr mxmlnwbr self-assigned this Dec 15, 2023
@mxmlnwbr
Copy link
Collaborator Author

@rschlaefli
Ready for review.
Should I also delete the ADMIN role?

@mxmlnwbr mxmlnwbr marked this pull request as ready for review December 15, 2023 10:01
@mxmlnwbr
Copy link
Collaborator Author

mxmlnwbr commented Dec 18, 2023

  • Remove Admin Role
  • Show ALL received feedbacks for dev role
  • Show basic as filter list (nothing else) ➡️ not implemented for now ➡️ testing if the information we get now is valuable enough (otherwise a lot of re-coding for maybe nothing)

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@mxmlnwbr mxmlnwbr merged commit d61fabd into dev Dec 18, 2023
2 checks passed
@mxmlnwbr mxmlnwbr deleted the devrole branch December 18, 2023 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants