-
Notifications
You must be signed in to change notification settings - Fork 1
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
SLR - SL-142 #82
Conversation
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.
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.
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 ); |
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.
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.
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.
Good idea.
This updates the code, where required, to support the customization features required by SLR in the SL-142 ticket.