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

#109 Implement update rolePermission #138

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

kennyg37
Copy link
Contributor

@kennyg37 kennyg37 commented Sep 23, 2024

Description

This pr adds a functionality to edit the role permissions of any pre-existing role in the database. If a role is created the role is given certain permissions like the permission to create other roles or the permission to use other various parts of the application. In previous versions the set roles couldn't be changed on the frontend and that is what this PR addresses.

Related issue

fixes #109

How to test

Run the project and try using the role and access functionality

cd atlp-devpulse-fn
git checkout ft-impl-update
npm run dev

Screenshots

Screenshot 2024-09-23 093929
Screenshot 2024-09-23 093947
Screenshot 2024-09-23 094012

Copy link

vercel bot commented Sep 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
atlp-devpulse-fn ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 26, 2024 10:37pm

@tysepa tysepa self-requested a review September 24, 2024 17:41
Copy link

codeclimate bot commented Sep 26, 2024

Code Climate has analyzed commit a1f43a5 and detected 0 issues on this pull request.

View more on Code Climate.

@kennyg37 kennyg37 changed the title Implement update rolePermission Implement update rolePermission #109 Sep 27, 2024
@kennyg37 kennyg37 changed the title Implement update rolePermission #109 #109 Implement update rolePermission Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BYTE SWAT BYTE SWAT Ready For Review Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

implementation of update rolePermission #90
3 participants