-
Notifications
You must be signed in to change notification settings - Fork 36
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
[SL-157] Support Uplink Licensing #2139
Conversation
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.
Approved, the wpunit tests are failing due to the older version of uplink. Once we get the version of uplink correct the tests should pass without issue.
The check tests locally use composer require stellarwp/uplink:dev-feat/fluent-form-builder --prefer-source
This adds the stellarwp/uplink library and configures it.
2724dc1
to
f397e06
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.
Same question about user cap check I left on the Uplink repository.
β¦-licensing-fix-uopz Uplink - Fix incorrect `uopz_redefine` arguments used within `set_const_value`
@@ -0,0 +1,174 @@ | |||
<?php |
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.
If all is required by the Uplink functionality is one file, why not move it to the root?
From ./Uplink/Controller.php
to ./Uplink.php
.
π« Ticket
SL-157
ποΈ Description
Provides a way for Uplink registered plugins to be displayed in License Tab.
It will hide legacy fields when it finds the same plugin being registered through Uplink
π₯ Artifacts
βοΈ Checklist
readme.txt
file.wpunit
orintegration
tests.wpunit
orintegration
tests.