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

Update Submodule #2619

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Update Submodule #2619

merged 1 commit into from
Jul 17, 2023

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Jul 17, 2023

Fix #2616
Close #2618

Verified 1:1 chats added a new API to Core, which DC Android now needs.

So far, we only updated the submodule on a release. I think we should also update it when DC Android needs a new API that was introduced in Core:

  1. To make builds "just work" for new contributors without having to checkout the correct commit in Core
  2. To make the preview builds of the CI here work.

Credits to @pjvleeuwen for bringing this problem up and exploring the different possible solutions for it!

Fix #2616
Closes #2618

Verified 1:1 chats added a new API to Core, which DC Android now needs.

So far, we only updated the submodule on a release. I think we should
also update it when DC Android needs a new API introduced in Core:

1. To make builds "just work" for new contributors without having to
   checkout the correct commit in Core
2. To make the preview builds of the CI here work.
@github-actions
Copy link

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

@Hocuri Hocuri merged commit 6b1527b into master Jul 17, 2023
2 checks passed
@Hocuri Hocuri deleted the hoc/update-submodule branch July 17, 2023 21:07
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.

reflect in documentation - deltachat-android submodule does not automatically update to newest master commit
2 participants