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

Avoid create duplicated access requests #252

Open
wallrony opened this issue May 23, 2022 · 1 comment · May be fixed by #278
Open

Avoid create duplicated access requests #252

wallrony opened this issue May 23, 2022 · 1 comment · May be fixed by #278
Assignees
@vassalo @wallrony
Labels
enhancement New feature or request

Comments

@wallrony
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Currently is possible to create more than one access request for a resource or role at the same time (before the first stales).

Describe the solution you'd like
Add a validation to verify if the user already has a pending request for that resource or role.
@wallrony wallrony added the enhancement New feature or request label May 23, 2022
wallrony added a commit that referenced this issue May 23, 2022
@wallrony @vassalo
Add validation for duplicated access requests (#252)
f259bf0 
Co-authored-by: vassalo <[email protected]>
wallrony added a commit that referenced this issue Aug 19, 2022
@wallrony @vassalo
Cherry-Pick Merge: Add validation for duplicated access requests (#252)
38268e5 
Co-authored-by: vassalo <[email protected]>
@wallrony wallrony linked a pull request Aug 19, 2022 that will close this issue
Open
wallrony pushed a commit that referenced this issue Aug 19, 2022
@vassalo @wallrony
Fix grant request helper unit test file existence verification (#252)
b201a4c 
Co-authored-by: vassalo <[email protected]>
@camposer
Copy link
Contributor

If we proceed to implement this, then the user should be enabled to read all the pending access requests. Maybe we can create a new command that gives you show access requests and then enable this feature.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@vassalo vassalo

@wallrony wallrony

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cherry-Pick Merge: Add validation for duplicated access requests (#252) strongdm/accessbot
3 participants
@camposer @vassalo @wallrony