Skip to content

Fix #176: Autosave and persistence for notes#177

Closed
Manu95021 wants to merge 6 commits intoHarshYadav152:mainfrom
Manu95021:fix-notes-unsaved
Closed

Fix #176: Autosave and persistence for notes#177
Manu95021 wants to merge 6 commits intoHarshYadav152:mainfrom
Manu95021:fix-notes-unsaved

Conversation

@Manu95021
Copy link

@Manu95021 Manu95021 commented Jan 27, 2026

This PR resolves issue #176, where users lose unsaved changes when refreshing the page or navigating to another section in the SaveBook application.
Changes Implemented:
Introduced autosave with debounce to persist note changes automatically.
Notes are now restored after page refresh or route change.
Added UI feedback to indicate save status (“Saving…” / “Saved”).
Improved state management to rely less on temporary in-memory state and more on persistent storage (localStorage and backend).
How to Test:
Edit or create a note.
Refresh the page → content should persist.
Navigate to a different route and return → content should persist.
Observe autosave feedback while typing.
Impact:
Prevents accidental loss of user data.
Improves reliability and user trust.
Lays groundwork for future offline support or further persistence enhancements.

@vercel
Copy link

vercel bot commented Jan 27, 2026

@Manu95021 is attempting to deploy a commit to the Harsh Yadav's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you @Manu95021 for your first PR to SaveBook!

We really appreciate your contribution 🙌

What happens next:

  • 🔍 Maintainers will review your PR
  • 🧪 Automated checks will run
  • ✨ Feedback may be shared if needed

Please confirm your PR includes:

  • ✔️ Clear summary of changes
  • ✔️ Linked issue (e.g., Fixes #123)
  • ✔️ Steps to test
  • ✔️ Screenshots (for UI changes)

📘 Contribution Standards:
👉 https://github.com/HarshYadav152/SaveBook/blob/main/CONTRIBUTING.md


💬 Stay Connected with Our Community

🎯 Discord (Official Communication)
For formal discussions, code reviews, and project updates:
👉 https://discord.gg/eMTePUK3gJ

💚 WhatsApp (Friendly Community)
For informal chats, quick help, and building friendships with contributors:
👉 https://chat.whatsapp.com/I8GYXd3mHlDCC2iXhNGeqV

🌟 Our Philosophy: We value both professional collaboration (Discord) and personal connections (WhatsApp). Join both to get the complete SS-Capture community experience!


Thanks for helping improve SaveBook 🚀
Let's build something amazing together! 💪

@Manu95021
Copy link
Author

please review it and merge

Copy link
Owner

@HarshYadav152 HarshYadav152 left a comment

Choose a reason for hiding this comment

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

@Manu95021
Do these changes

  1. Remove package-lock.json from repo root.
  2. Revert any changes make to README.md
  3. Here you write everything else remain unchanged
    • by this you removed required code from Notes.js
    • You just have to make changes to file with your implementation but do not remove any code from it until & unless it is become redundant.
image

keep your branch is up to date with main.

Copy link
Owner

@HarshYadav152 HarshYadav152 left a comment

Choose a reason for hiding this comment

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

@Manu95021 Do these changes

  1. Move useEffect from the bottom of the code to inside functional component Notes()
  2. Also noteContent is not defined

@HarshYadav152
Copy link
Owner

@Manu95021 Have you updated your branch with request changes.

@HarshYadav152
Copy link
Owner

@Manu95021
useEffect must be used inside a component.

@HarshYadav152
Copy link
Owner

Close as requested by assignee

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants