Skip to content

Feat: Added notification feedback when using Export as File#976

Closed
Audatic07 wants to merge 9 commits intoCircuitVerse:mainfrom
Audatic07:add-export-feedback#975
Closed

Feat: Added notification feedback when using Export as File#976
Audatic07 wants to merge 9 commits intoCircuitVerse:mainfrom
Audatic07:add-export-feedback#975

Conversation

@Audatic07
Copy link
Contributor

@Audatic07 Audatic07 commented Feb 22, 2026

Fixes #975

Describe the changes you have made in this PR -

Added notification pop-up when using download as file, in both tauri and web versions. The tauri version lists the whole file path of the download.

Screenshots of the UI changes (If any) -

Screenshot 2026-02-22 234533 Screenshot 2026-02-22 234609

Code Understanding and AI Usage

Did you use AI assistance (ChatGPT, Claude, Copilot, etc.) to write any part of this code?

  • No, I wrote all the code myself
  • Yes, I used AI assistance (continue below)

If you used AI assistance:

  • I have reviewed every single line of the AI-generated code
  • I can explain the purpose and logic of each function/component I added
  • I have tested edge cases and understand how the code handles them
  • I have modified the AI output to follow this project's coding standards and conventions

Explain your implementation approach:


Checklist before requesting a review

  • I have added proper PR title and linked to the issue
  • I have performed a self-review of my code
  • I can explain the purpose of every function, class, and logic block I added
  • I understand why my changes work and have tested them thoroughly
  • I have considered potential edge cases and how my code handles them
  • If it is a core feature, I have added thorough tests
  • My code follows the project's style guidelines and conventions

Note: Please check Allow edits from maintainers if you would like us to assist in the PR.

Summary by CodeRabbit

  • New Features

    • Project export now displays a success message confirming the file location upon save completion.
    • Project recovery provides enhanced user feedback with clearer error messages and validation warnings.
  • Bug Fixes

    • Improved project recovery with robust validation of recovered data and better error handling for corrupted or invalid recovery files.

@netlify
Copy link

netlify bot commented Feb 22, 2026

Deploy Preview for circuitverse ready!

Name Link
🔨 Latest commit 77a7ad3
🔍 Latest deploy log https://app.netlify.com/projects/circuitverse/deploys/699b4f6b19e0b400085f9f1c
😎 Deploy Preview https://deploy-preview-976--circuitverse.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 46 (🟢 up 1 from production)
Accessibility: 66 (no change from production)
Best Practices: 92 (no change from production)
SEO: 82 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 22, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This PR implements user feedback notifications for file export operations and enhances project recovery error handling. Changes include: making downloadFile functions asynchronous to return the download directory path; adding success messages in ExportProject.vue showing the saved filename and destination; rewriting recoverProject with comprehensive validation, error handling, and user feedback for localStorage-based recovery scenarios; and applying these modifications consistently across both src/ and v1/ versions of the codebase.

Possibly related PRs

  • PR #380: Modifies the download helper functions and their usage in ExportProject.vue with matching async/return semantics changes.
  • PR #800: Changes the export workflow by removing legacy SaveAs.js and migrating behavior to ExportProject.vue, which this PR directly updates.

Suggested labels

GSOC'24

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Audatic07 Audatic07 closed this Feb 22, 2026
@Audatic07 Audatic07 deleted the add-export-feedback#975 branch February 22, 2026 19:02
@Audatic07
Copy link
Contributor Author

Made this branch layered from another unmerged branch accidentally, now making a new PR. Sorry :_)

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.

Feature: Feedback notification popup when completing Export as File

1 participant