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

Special offer #968

Merged
merged 11 commits into from
Aug 28, 2024
Merged

Special offer #968

merged 11 commits into from
Aug 28, 2024

Conversation

pereirinha
Copy link
Member

Approach

  • Adds the special offer

Copy link
Collaborator

@tharsheblows tharsheblows left a 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.

php/class-special-offer.php Outdated Show resolved Hide resolved
@pereirinha pereirinha marked this pull request as ready for review August 28, 2024 15:46
Copy link
Collaborator

@tharsheblows tharsheblows left a 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
);
Copy link
Collaborator

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">',
Copy link
Collaborator

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.

@pereirinha pereirinha merged commit 06bcbf5 into develop Aug 28, 2024
4 checks passed
@pereirinha pereirinha deleted the feature/29-offer branch September 3, 2024 14:12
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