-
Notifications
You must be signed in to change notification settings - Fork 28
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
Special offer #968
Special offer #968
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.
This is in draft but I have one comment to fix! Otherwise it works well for me.
Great catch ;) Co-authored-by: Mary (JJ) Jay <[email protected]>
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.
There is one comment and that's it! It's very small.
'request_description' => __( 'Please, provide more details on your request, and if possible, attach a System Report', 'cloudinary' ), | ||
$args = wp_parse_args( | ||
$args, | ||
$default_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.
👏 praise
I like this approach!
return sprintf( | ||
// translators: The HTML markup. | ||
__( 'We’re compatible with most other plugins so we expect it to work absolutely fine. If you do have any issues, please %1$scontact our support team%2$s who will help resolve your issue.', 'cloudinary' ), | ||
'<a href="' . Utils::get_support_link() . '" target="_blank" rel="noopener noreferrer">', |
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.
🥜 nitpick
Since there's translated text in the support link, this probably should be escaped.
Approach