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

fix(email): Remove 'the' from some transactional emails #2796

Merged
merged 1 commit into from
Dec 22, 2023
Merged

Conversation

alfl
Copy link
Contributor

@alfl alfl commented Dec 22, 2023

Description

Two transactional emails -- purchase successful and purchase failure -- referred to "the Rollup ID". This removes the article "the" so that branding is more consistent.

Related Issues

  • NA

Testing

NA

Checklist

  • I have read the CONTRIBUTING guidelines
  • [na] I have tested my code (manually and/or automated if applicable)
  • [na] I have updated the documentation (if necessary)

Two transactional emails -- purchase successful and purchase failure -- referred to "the Rollup ID". This removes the article "the" so that branding is more consistent.
@alfl alfl added the bug Indicates an unexpected problem or unintended behavior label Dec 22, 2023
@alfl alfl added this to the Future milestone Dec 22, 2023
@alfl alfl self-assigned this Dec 22, 2023
@szkl szkl merged commit c18cf1b into main Dec 22, 2023
14 checks passed
@szkl szkl deleted the fix/emails branch December 22, 2023 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants