Skip to content

Feature: Added Undo option after transaction deletes#115

Open
sharon44-ham wants to merge 14 commits into
TanmayKalra09:mainfrom
sharon44-ham:feature/undoTransaction
Open

Feature: Added Undo option after transaction deletes#115
sharon44-ham wants to merge 14 commits into
TanmayKalra09:mainfrom
sharon44-ham:feature/undoTransaction

Conversation

@sharon44-ham
Copy link
Copy Markdown

🚀 Pull Request

Feature: Added Undo option after transaction deletes

📋 Description

Added Undo functionality for deleted transactions.
Stores last deleted transaction in state and restores it if user clicks Undo within few seconds.

Please include a summary of the change and which issue is fixed (if any).

Fixes #<12>

🔍 Type of Change

  • Bug fix 🐞
  • [ ✅,] New feature ✨
  • Documentation update 📝
  • Refactoring or code improvement ♻️
  • Other (please describe):

🙋 Your Details

🧪 How Has This Been Tested?

Manual Testing

  1. Added a few transactions (Income and Expense).
  2. Deleted a transaction and verified that the "Undo" option appeared.
  3. Clicked "Undo" within 5 seconds → Transaction was restored successfully.
  4. Waited for the Undo timeout → Transaction remained deleted.
  5. Verified data persistence after page refresh (localStorage updates correctly).
  • Edge Cases
    • Deleted multiple transactions quickly → Only last deleted transaction was restored.
    • Checked that Undo does not create duplicates.

-Screenshot
Screenshot from 2025-07-26 12-49-03

✅ Checklist

  • [✅ ] I have read the contributing guidelines.
  • [ ✅] I have followed the code style and linting rules.
  • [ ✅] I have added tests or explained why not.
  • [✅ ] I have updated documentation (if needed).
  • [ ✅] My changes do not introduce any known security issues or vulnerabilities.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 26, 2025

@sharon44-ham is attempting to deploy a commit to the Tanmay Kalra's projects Team on Vercel.

A member of the Team first needs to authorize it.

@TanmayKalra09
Copy link
Copy Markdown
Owner

@sharon44-ham please resolve the conflicts

@sharon44-ham
Copy link
Copy Markdown
Author

@sharon44-ham please resolve the conflicts

Hey resolved the conflicts.
can you please look into it now and update me if anything else to be done from my side.Thank you!

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 27, 2025

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

Name Status Preview Comments Updated (UTC)
smart-log ❌ Failed (Inspect) Jul 27, 2025 5:22pm

@TanmayKalra09
Copy link
Copy Markdown
Owner

@sharon44-ham your code contains some error, please resolve it.

@sharon44-ham
Copy link
Copy Markdown
Author

@sharon44-ham your code contains some error, please resolve it.

hey i cant view errorlog in vercel,i have cleared all errors i got locally by running npm run build however yet it shows some checks were not successful:(
would appreciate any sort of guidance or help.Thank you!

Copy link
Copy Markdown
Collaborator

@Tanziha-Sheikh Tanziha-Sheikh left a comment

Choose a reason for hiding this comment

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

Hello @sharon44-ham

I have reviewed your PR and find out that you have done lots of changes in the files while working on undo transaction. I don't think that all these file need the changes or maybe the changes made while resolving the conflicts.

Please explain why did you update the react and react dom package.

IMG_20250728_032742

Directly upgrading the react version will affect the overall project and some functionality stops working. It's not required here.

Also, in the Dashboard.jsx lots of code are removed. Please check it and make sure that someone else code doesn't removed by you even unintentionally.

IMG_20250728_032908

Please check the above points as soon as possible and update us.

Thanks
@Tanziha-Sheikh
CC- @TanmayKalra09

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.

3 participants