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

Showing a warning dialog when copying/moving the ZIM chunks in PS version. #4170

Merged
merged 5 commits into from
Jan 19, 2025

Conversation

MohitMaliFtechiz
Copy link
Collaborator

@MohitMaliFtechiz MohitMaliFtechiz commented Jan 8, 2025

Fixes #4045

  • Showing a warning dialog when copying/moving the ZIM chunks in the PS version.
  • Improved the copying/moving of ZIM chunks, and improved the validation of ZIM chunks.
  • The warning dialog will only show for the ZIM chunks, for normal ZIM files it will directly open the ZIM file in the reader like it was working.
screen-20250109-185127.mp4

@MohitMaliFtechiz MohitMaliFtechiz marked this pull request as draft January 8, 2025 12:53
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 31.57895% with 13 lines in your changes missing coverage. Please review.

Project coverage is 56.86%. Comparing base (283d845) to head (a3f8793).

Files with missing lines Patch % Lines
...kiwix/kiwixmobile/core/utils/dialog/KiwixDialog.kt 0.00% 6 Missing ⚠️
...le/nav/destination/library/LocalLibraryFragment.kt 50.00% 3 Missing and 2 partials ⚠️
...ile/nav/destination/library/CopyMoveFileHandler.kt 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4170      +/-   ##
============================================
- Coverage     56.93%   56.86%   -0.08%     
  Complexity     1518     1518              
============================================
  Files           313      313              
  Lines         13402    13417      +15     
  Branches       1666     1666              
============================================
- Hits           7631     7630       -1     
- Misses         4614     4633      +19     
+ Partials       1157     1154       -3     

☔ 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 9, 2025 17:12
* Now all the ZIM chunks are copying/moving correctly.
* Improved the warning message to inform user what should they do after copying/moving all the ZIM chunks.
* Implement a feature to directly view the copied/moved ZIM chunks by clicking on the button in warning dialog.
…rify the ZIM chunks are copied/moved properly.
@kelson42 kelson42 merged commit 1949d29 into main Jan 19, 2025
28 checks passed
@kelson42 kelson42 deleted the Fixes#4045 branch January 19, 2025 15:20
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.

Does ZIM file move/copy works fine with ZIM chunks?
3 participants