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

Feat/permission phase 2 #2615

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

Feat/permission phase 2 #2615

wants to merge 27 commits into from

Conversation

akhilmhdh
Copy link
Member

Description 📣

  1. Bring back revereted changes for permission phase 2
  2. Resolved an edge case that was in project folder getting created without any actions.
  3. Ensured tf works smooth

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

Tests 🛠️

# Here's some code block to paste some code snippets

akhilmhdh and others added 24 commits October 23, 2024 16:44
@akhilmhdh
Revert "Revert "Permission phase 2""
f487d76 
This reverts commit 8b9244b.
@akhilmhdh
Revert "Revert "feat: added filter folder to remove read only in migr…
e654077 
…ation""

This reverts commit 78f668b.
@akhilmhdh
fix: resolved edge case in folder empty action
678a0af
@akhilmhdh
feat: resolved identity privilege removed folder, dynamic secret and …
832af06 
…secret import permission
@akhilmhdh
feat: added backend endpoint for new identitiy additional privilege p…
3e4b763 
…ermission system
@akhilmhdh
feat: update user additional privilege router to new one
e56bd08
@akhilmhdh
feat: made v2 project role router use projectid instead of projectslug
761b3bd
@akhilmhdh
feat: role list section completed with delete
6d7f10f
@akhilmhdh
feat: added role modify popup
ae9addb
@akhilmhdh
feat: completed new user detail page screen
2039565
@akhilmhdh
feat: completed identity project detail screen
35c2c7f
@akhilmhdh
feat: resolved role issue in ui
7d68076
@akhilmhdh
feat: completed all nit changes in review
45d23ea
@scott-ray-wilson @akhilmhdh
docs: add missing periods, standardize ID and fix typos to api constants
0738890
@scott-ray-wilson @akhilmhdh
docs: add example temporal duration for temporary range
d04893c
@akhilmhdh
feat: removed form in member and identity table, and added ellipsis f…
4e44981 
…or noting icons
@scott-ray-wilson @akhilmhdh
resolve rebase
d5e95b6
@scott-ray-wilson @akhilmhdh
improvement: make identity add priv edit consistent with member
6878be9
@scott-ray-wilson @akhilmhdh
improvement: fix variable name
a0f47c9
@scott-ray-wilson @akhilmhdh
improvement: add priv conflict info and improve back button
7a4ff72
@akhilmhdh
feat: added missing remove identity, user handler, changed title to d…
5948084 
…uration for access type
@akhilmhdh
feat: removed console and resolved drift in tf due to folder removing…
2b1eac5 
… in v1
@akhilmhdh
feat: resolved ts fail in license check
904bde5
@akhilmhdh
feat: resolved merge issues
a4e90d7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maidul98 maidul98 Awaiting requested review from maidul98

At least 1 approving review is required to merge this pull request.

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
@akhilmhdh @scott-ray-wilson