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

Remove prepareMsg (and OUT_PREPARING message state) #3468

Merged
merged 3 commits into from
Dec 11, 2024

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Dec 4, 2024

This PR removes prepareMsg, which put messages into the OUT_PREPARING state and was used while videos were recoded:

  • It was buggy because when you forwarded a message while it was InPreparation, or when the app was killed while a message is InPreparation, the message would stay InPreparation forever.
  • Android is the only UI using this InPreparation (according to @r10s, I didn't check this myself), so we can simplify some things in core, which will also make it easier to deduplicate blob files.

Copy link

github-actions bot commented Dec 4, 2024

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

@Hocuri Hocuri force-pushed the hoc/remove-inpreparation branch from 364306a to 0d75504 Compare December 6, 2024 12:15
@Hocuri Hocuri force-pushed the hoc/remove-inpreparation branch from 0d75504 to 2bd1502 Compare December 6, 2024 12:15
@Hocuri Hocuri requested review from r10s, link2xt and adbenitez December 6, 2024 12:19
@Hocuri Hocuri changed the title [WIP] Start removing InPreparation message state [WIP] Start removing prepareMsg (and OUT_PREPARING message state) Dec 6, 2024
@Hocuri Hocuri changed the title [WIP] Start removing prepareMsg (and OUT_PREPARING message state) Remove prepareMsg (and OUT_PREPARING message state) Dec 6, 2024
Copy link

github-actions bot commented Dec 6, 2024

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

Copy link
Contributor

@link2xt link2xt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, did not test.

From the code it seems there is no way to cancel recoding in progress, right?

@r10s
Copy link
Member

r10s commented Dec 7, 2024

isIncreation() can be removed from dc_wrapper.c and from DcMsg.java as well - it was not used recently anyways, but now soon definetely pointness and removed from core

@link2xt
Copy link
Contributor

link2xt commented Dec 11, 2024

Corresponding core PR removing now unused API: deltachat/deltachat-core-rust#6319

@Hocuri
Copy link
Collaborator Author

Hocuri commented Dec 11, 2024

From the code it seems there is no way to cancel recoding in progress, right?

Yes, right. As long as we don't get feature requests for it, I think it's fine not to have it.

@Hocuri Hocuri merged commit 963327d into main Dec 11, 2024
2 checks passed
@Hocuri Hocuri deleted the hoc/remove-inpreparation branch December 11, 2024 12:53
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.

4 participants