Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed: When a note is deleted, the corresponding file remained in storage. #4195

Merged
merged 2 commits into from
Jan 31, 2025

Conversation

MohitMaliFtechiz
Copy link
Collaborator

@MohitMaliFtechiz MohitMaliFtechiz commented Jan 29, 2025

Fixes #4194

  • The note file is now deleted from storage when the user selects multiple notes and deletes them from the "Notes" screen.
  • Improved note deletion in the "Note Dialog." Previously, when deleting a note from the AddNoteDialog on the Notes screen, the file was removed from storage, but its entry remained in the database, causing the note to still appear on the "Notes" screen after deletion. This issue has now been fixed.
  • Added the UI test cases for both scenarios to avoid future errors.
screen-20250129-144837.mp4

@MohitMaliFtechiz MohitMaliFtechiz marked this pull request as draft January 29, 2025 09:23
Copy link

codecov bot commented Jan 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.83%. Comparing base (3da8c40) to head (561af8f).
Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4195      +/-   ##
============================================
+ Coverage     56.75%   56.83%   +0.08%     
- Complexity     1532     1537       +5     
============================================
  Files           313      313              
  Lines         13506    13513       +7     
  Branches       1694     1695       +1     
============================================
+ Hits           7665     7680      +15     
+ Misses         4675     4666       -9     
- Partials       1166     1167       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MohitMaliFtechiz MohitMaliFtechiz marked this pull request as ready for review January 29, 2025 11:17
…rage.

* The note file is now deleted from storage when the user selects multiple notes and deletes them from the "Notes" screen.
* Improved note deletion in the "Note Dialog." Previously, when deleting a note from the `AddNoteDialog` on the Notes screen, the file was removed from storage, but its entry remained in the database, causing the note to still appear on the "Notes" screen after deletion. This issue has now been fixed.
@kelson42 kelson42 merged commit e61e3e8 into main Jan 31, 2025
31 checks passed
@kelson42 kelson42 deleted the Fixes#4194 branch January 31, 2025 14:32
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.

When we delete the note the corresponding file remains saved in the file storage.
3 participants