Skip to content

feature: add snapshot history panel to recover data after reset#90

Open
shrutisharma-sh wants to merge 3 commits into
Mohit-368:mainfrom
shrutisharma-sh:feature/reset-history-snapshots
Open

feature: add snapshot history panel to recover data after reset#90
shrutisharma-sh wants to merge 3 commits into
Mohit-368:mainfrom
shrutisharma-sh:feature/reset-history-snapshots

Conversation

@shrutisharma-sh
Copy link
Copy Markdown

@shrutisharma-sh shrutisharma-sh commented May 12, 2026

Closes #85
Screenshot (281)

What this does

  • Saves a snapshot to localStorage before every reset
  • Adds a History button that opens a panel of up to 15 timestamped snapshots
  • Each snapshot has a Restore button that reloads the form with that data
  • Adds a confirmation prompt before reset so accidental clicks don't wipe data

Why

One misclick on Reset wiped 10–15 minutes of work with no recovery. This fixes that.

Changes

  • readmeforge.js — added pushToHistory, restoreFromHistory, openHistoryPanel, closeHistoryPanel, updated resetAll()
  • index.html — added History button and modal panel

@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for makeareadme ready!

Name Link
🔨 Latest commit de80471
🔍 Latest deploy log https://app.netlify.com/projects/makeareadme/deploys/6a04b6a411cf1a0008246356
😎 Deploy Preview https://deploy-preview-90--makeareadme.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@shrutisharma-sh
Copy link
Copy Markdown
Author

Hi @Mohit-368

I have fixed this issue. , PR #90 Here's what I changed:

  • Added pushToHistory() to snapshot localStorage before every reset
  • Added a History button that opens a panel with up to 15
    timestamped snapshots, each with a Restore button
  • Added a confirmation prompt to resetAll() to prevent accidental wipes
  • Zero new dependencies built on the existing localStorage architecture

Please review my PR. Thank you .

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.

## Issue: Reset button clears all data with no way to recover it

1 participant