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

Fix WordPressComRestApi.GETData #746

Merged
merged 4 commits into from
Mar 8, 2024
Merged

Fix WordPressComRestApi.GETData #746

merged 4 commits into from
Mar 8, 2024

Conversation

crazytonyli
Copy link
Contributor

Description

The GetData function was implemented incorrectly in #720, which caused two endpoint implementations failing to parse HTTP response body.

Testing Details

See the new unit tests.


  • Please check here if your pull request includes additional test coverage.
  • I have considered if this change warrants release notes and have added them to the appropriate section in the CHANGELOG.md if necessary.

@crazytonyli crazytonyli requested review from mokagio and guarani March 8, 2024 01:16
@crazytonyli crazytonyli mentioned this pull request Mar 8, 2024
2 tasks
Copy link
Contributor

@guarani guarani left a comment

Choose a reason for hiding this comment

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

I tested it locally and confirmed feature announcements are working again 🙇

I hadn't noticed the Zendesk issue, thanks for fixing that as well.

@guarani guarani merged commit 61d077c into trunk Mar 8, 2024
9 checks passed
@guarani guarani deleted the fix-com-rest-api-get-data branch March 8, 2024 01:32
@guarani
Copy link
Contributor

guarani commented Mar 8, 2024

Merging as asked for in p1709860929336219/1709766129.730749-slack-C04PWEZSYFL. @mokagio, can this please be included in the 24.4 beta? It fixes feature announcements and possibly an issue with Zendesk determining the user's plan.

@guarani guarani mentioned this pull request Mar 8, 2024
2 tasks
crazytonyli added a commit to wordpress-mobile/WordPress-iOS that referenced this pull request Mar 8, 2024
Including a bug fix to getting feature announcement and zendesk
metadata. See
wordpress-mobile/WordPressKit-iOS#746 and
wordpress-mobile/WordPressKit-iOS#747.

See the full 14.0.0 - 14.0.1 changes [here][diff].

[diff]: wordpress-mobile/WordPressKit-iOS@14.0.0...14.0.1
crazytonyli added a commit to wordpress-mobile/WordPress-iOS that referenced this pull request Mar 8, 2024
Including a bug fix to getting feature announcement and zendesk
metadata. See
wordpress-mobile/WordPressKit-iOS#746 and
wordpress-mobile/WordPressKit-iOS#747.

See the full 14.0.0 - 14.0.1 changes [here][diff].

[diff]: wordpress-mobile/WordPressKit-iOS@14.0.0...14.0.1
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