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

Adapt more emails for CE #4644

Closed
wants to merge 5 commits into from
Closed

Conversation

ruslandoga
Copy link
Contributor

@ruslandoga ruslandoga commented Oct 2, 2024

Changes

This WIP PR continues #4496 and updates other emails / UI.

I plan to split it into smaller, targeted PRs, this one is mostly an exploration into how to make sure no trial / EE messages end up in CE by relying on the compiler.

Related: #2213 (comment) #3853 (comment)

Tests

  • Automated tests have been added

Changelog

  • Entry has been added to 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 changed the title don't mention trial in site_setup_help email Adapt emails for CE Oct 2, 2024
@ruslandoga ruslandoga changed the title Adapt emails for CE Adapt more emails for CE Oct 2, 2024
@ruslandoga ruslandoga added the self-hosting Anything self-hosted label Oct 2, 2024
@ruslandoga
Copy link
Contributor Author

Closing in favor of a smaller PR: #4668

@ruslandoga ruslandoga closed this Oct 14, 2024
@ruslandoga ruslandoga deleted the remove-trial-mentions-from-emails branch October 14, 2024 14:12
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.

1 participant