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

Remove trial mentions from site emails in CE #4668

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

ruslandoga
Copy link
Contributor

@ruslandoga ruslandoga commented Oct 10, 2024

Changes

This PR continues #4496 and updates two other site setup emails.

Related: #2213 (comment)

Tests

  • Automated tests have been added

Changelog

  • This PR does not make a user-facing change

Documentation

  • This change does not need a documentation update

Dark mode

  • This PR does not change the UI

@ruslandoga ruslandoga marked this pull request as ready for review October 10, 2024 15:06
@ruslandoga ruslandoga requested a review from a team October 10, 2024 15:06
@ruslandoga ruslandoga mentioned this pull request Oct 14, 2024
5 tasks
@ruslandoga ruslandoga added the self-hosting Anything self-hosted label Oct 14, 2024
@ruslandoga ruslandoga force-pushed the remove-trial-mention-from-site-setup-email branch from ef6da73 to 4d04718 Compare October 16, 2024 04:01
<%= if Plausible.ee?() do %>
your free 30-day trial of
<% end %>
Plausible, a simple and privacy-friendly website analytics tool. <br /><br />
Copy link
Contributor Author

@ruslandoga ruslandoga Oct 16, 2024

Choose a reason for hiding this comment

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

Trial upgrade email uses similar wording (just "Plausible") but it's not really used in CE:

Thanks for exploring Plausible, a simple and privacy-friendly alternative to Google Analytics. Your free 30-day trial is ending <%= @day %>, but you can keep using Plausible by upgrading to a paid plan.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'd like to double-check if it's OK to use "Plausible" in CE

@ukutaht

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
self-hosting Anything self-hosted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants