-
Notifications
You must be signed in to change notification settings - Fork 27
fix: disable input elements in PRO upsell #1096
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
324d558
to
d7533d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR disables input elements in the conditions control for non-PRO users to prevent interaction with PRO-only features. The change improves the user experience by making the upsell state more explicit through disabled form controls.
- Adds early return in the
onChangeCondtion
function for non-PRO users - Disables all form controls (SelectControl and TextControl) when PRO features are not available
- Refactors inline PRO check to use a consistent
isPro
variable
Plugin build for 7c3934d is ready 🛎️!
Note You can preview the changes in the Playground |
3ed86b2
to
7c3934d
Compare
Summary
Will affect visual aspect of the product
NO
Screenshots
Test instructions
Check before Pull Request is ready:
Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/860