Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Fix a regression in plan flag logic - #578

Merged
guarani merged 3 commits into
trunkfrom
fix/plan-flag
Feb 17, 2023
Merged

Fix a regression in plan flag logic#578
guarani merged 3 commits into
trunkfrom
fix/plan-flag

Conversation

@guarani

@guarani guarani commented Feb 17, 2023

Copy link
Copy Markdown
Contributor

Description

Fixes #579

Testing Details

Testing instructions can be found in wordpress-mobile/WordPress-iOS#20163.


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

Comment thread WordPressKit/RemoteBlog.swift Outdated
@guarani
guarani marked this pull request as ready for review February 17, 2023 22:54

@twstokes twstokes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@guarani

guarani commented Feb 17, 2023

Copy link
Copy Markdown
Contributor Author

LGTM @guarani! Tests passed for me in the WPiOS PR. 🚀

Thank you, kind sir! 🙇

@guarani
guarani merged commit 85336d9 into trunk Feb 17, 2023
@guarani
guarani deleted the fix/plan-flag branch February 17, 2023 23:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect logic for querying if blog's plan is free

2 participants