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 from
Feb 17, 2023
Merged

Fix a regression in plan flag logic #578

guarani merged 3 commits into from
Feb 17, 2023

Conversation

@guarani
Copy link
Contributor

@guarani guarani commented Feb 17, 2023 •
edited
Loading

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.

@guarani guarani mentioned this pull request Feb 17, 2023
Merged
4 tasks
twstokes
twstokes reviewed Feb 17, 2023
View reviewed changes
@guarani guarani marked this pull request as ready for review February 17, 2023 22:54
twstokes
twstokes approved these changes Feb 17, 2023
View reviewed changes
Copy link
Contributor

@twstokes twstokes left a comment

Choose a reason for hiding this comment

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

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

@guarani
Copy link
Contributor Author

guarani commented Feb 17, 2023

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.

Reviewers

1 more reviewer

@twstokes twstokes twstokes approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Incorrect logic for querying if blog's plan is free

3 participants

@guarani @twstokes