Skip to content

Adding Action Confirmations#400

Open
adityapat24 wants to merge 4 commits intomainfrom
397-dev-adding-more-action-confirmations
Open

Adding Action Confirmations#400
adityapat24 wants to merge 4 commits intomainfrom
397-dev-adding-more-action-confirmations

Conversation

@adityapat24
Copy link
Copy Markdown
Contributor

ℹ️ Issue

Closes #397

📝 Description

Added the action confirmation modal based on the discussed user actions

Briefly list the changes made to the code:

  1. Added modal when editing and saving grant
  2. Added to delete user profile picture
  3. Added for changing user personal information

✔️ Verification

What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.

Frontend Testing
image
image
image
image
image

@adityapat24 adityapat24 linked an issue Apr 9, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@janekamata janekamata left a comment

Choose a reason for hiding this comment

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

Lgtm! Tested myself by going through the different popups and everything seemed to be working

Copy link
Copy Markdown
Collaborator

@prooflesben prooflesben left a comment

Choose a reason for hiding this comment

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

This pr is going to be extended to fit to all state changing actions since that's what hessann said in the client meeting bc old people are old people

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DEV Adding more action confirmations

3 participants