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

SLR - SL-142 #82

Merged
merged 10 commits into from
Sep 4, 2024
Merged

SLR - SL-142 #82

merged 10 commits into from
Sep 4, 2024

Conversation

lucatume
Copy link
Contributor

@lucatume lucatume commented Aug 19, 2024

This updates the code, where required, to support the customization features required by SLR in the SL-142 ticket.

CleanShot 2024-09-03 at 17 55 04

Add a further filter to control the connection URL text by not only hook
prefix, that will be unique per plugin, but by slug (unique by feature
in the context of a plugin) as well.
@lucatume lucatume self-assigned this Aug 19, 2024
@lucatume lucatume marked this pull request as ready for review September 2, 2024 09:45
@lucatume lucatume requested a review from tarecord September 2, 2024 09:45
Copy link
Member

@dpanta94 dpanta94 left a comment

Choose a reason for hiding this comment

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

Looks good! Just an apply filters i'd like us we implement and a question!

if ( $this->resource->is_using_oauth() ) {
ob_start();

if ( $this->resource->oauth_requires_license_key() ) {
echo $this->view->render( self::VIEW, $args );
Copy link
Member

@dpanta94 dpanta94 Sep 3, 2024

Choose a reason for hiding this comment

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

I would somehow like to apply_filters( 'stellarwp/uplink/' . Config::get_hook_prefix() . '/license_field_html', ... to this as well. From an external developer's perspective utilizing the filter should modify the html regardless if the license field is printed as part of the oauth or not.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good idea.

tests/wpunit/Admin/Fields/FieldTest.php Show resolved Hide resolved
@lucatume lucatume merged commit 11b5e6d into main Sep 4, 2024
13 checks passed
@lucatume lucatume deleted the slr/sl-142 branch September 4, 2024 09:48
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