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

File deduplication, Android part #3513

Merged
merged 7 commits into from
Jan 27, 2025
Merged

File deduplication, Android part #3513

merged 7 commits into from
Jan 27, 2025

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Dec 27, 2024

Depends on chatmail/core#6332.

Remaining bugs:

  • When sharing an image to DC (or when drafting and re-entering a chat), the dimensions aren't calculated correctly Fixed thanks to adb
  • When sharing a video to DC (or when drafting and re-entering a chat) and then clicking on the draft to preview it, I only get a black screen Works fine with feat: Keep file extension on deduplicated files chatmail/core#6463

@Hocuri Hocuri force-pushed the hoc/file-deduplication branch from 9750e7b to d5503f6 Compare January 6, 2025 18:23
@Hocuri Hocuri force-pushed the hoc/file-deduplication branch from e3ddc7b to 3ef45d2 Compare January 8, 2025 09:46
@Hocuri Hocuri force-pushed the hoc/file-deduplication branch 3 times, most recently from 73a9f89 to e213b6a Compare January 21, 2025 13:36
@Hocuri Hocuri force-pushed the hoc/file-deduplication branch from e213b6a to 7daf12f Compare January 22, 2025 13:58
@Hocuri Hocuri changed the title [WIP] File deduplication, Android part File deduplication, Android part Jan 22, 2025
@Hocuri Hocuri requested review from r10s and adbenitez January 22, 2025 16:18
@Hocuri
Copy link
Collaborator Author

Hocuri commented Jan 22, 2025

Probably I should bump the core submodule in this PR so that building the preview APK succeeds? (it's failing because Android now depends on the new dc_msg_set_file_and_deduplicate() API)

Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@adbenitez adbenitez force-pushed the hoc/file-deduplication branch from 2286fb5 to a5f80f5 Compare January 27, 2025 16:42
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@Hocuri Hocuri merged commit c6b8905 into main Jan 27, 2025
2 checks passed
@Hocuri Hocuri deleted the hoc/file-deduplication branch January 27, 2025 17:40
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.

2 participants