Skip to content

Chore: Adapt requests for saved post and forwarded messages endpoints #530

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

Conversation

julian-wls
Copy link
Contributor

@julian-wls julian-wls commented Apr 1, 2025

Problem Description

With this PR on the Artemis repo, the endpoint structure will be changed for all saved posts and forwarded messages requests.

Changes

This PR applies the changes to the Android app.

Steps for testing

  1. Go into any channel on a test server that has new API structure applied
  2. Make sure forwarding posts are working as expected
  3. Make sure saving posts for later works as expected

@julian-wls julian-wls self-assigned this Apr 1, 2025
@julian-wls julian-wls added blocked This issue is blocked by another issue. Please mention the blocking issue api calls Attention when merging because of the recent 8.0.0 breaking changes ready for review This PR can be reviewed and removed blocked This issue is blocked by another issue. Please mention the blocking issue labels Apr 1, 2025
@julian-wls julian-wls marked this pull request as ready for review April 3, 2025 06:39
@julian-wls julian-wls requested a review from FelberMartin April 3, 2025 06:46
@julian-wls julian-wls requested a review from FelberMartin April 8, 2025 07:46
@FelberMartin FelberMartin added ready to merge This PR can be merged and removed ready for review This PR can be reviewed labels Apr 8, 2025
@FelberMartin FelberMartin merged commit 3503a01 into develop Apr 8, 2025
5 checks passed
@FelberMartin FelberMartin deleted the chore/communication/adapt-forward-messages-and-saved-posts-endpoint branch April 8, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api calls Attention when merging because of the recent 8.0.0 breaking changes ready to merge This PR can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants