Skip to content

Rc/update member invite copy #41

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.

Sign up for GitHub

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
+22 −11
Open

Rc/update member invite copy #41

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2f4dc81
Revise member invite email content
rahulchhabria Oct 7, 2025
15a4d03
Revise member invite email content and layout
rahulchhabria Oct 7, 2025
7825f72
:hammer_and_wrench: apply pre-commit fixes
getsantry[bot] Oct 7, 2025
3bbc03d
formatting fixes
rahulchhabria Oct 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 13 additions & 5 deletions src/sentry/templates/sentry/emails/member-invite.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,22 @@
{% load sentry_helpers %}

{% block main %}
<h3>You've been invited to Sentry</h3>
<p>Your teammates at <strong>{{ organization.name|sanitize_periods }}</strong> are using Sentry to track and debug software errors.</p>
<p>He there,</p>
Copy link

Choose a reason for hiding this comment

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

[BestPractice]

Typo in greeting: "He there," should be "Hey there," to match the plain text version and standard greeting format.

Suggested change
<p>He there,</p>
<p>Hey there,</p>

⚡ Committable suggestion

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Context for Agents 
[**BestPractice**]

Typo in greeting: "He there," should be "Hey there," to match the plain text version and standard greeting format.

```suggestion
    <p>Hey there,</p>
```

⚡ **Committable suggestion**

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

File: src/sentry/templates/sentry/emails/member-invite.html
Line: 7

<p>Your teammates at <strong>{{ organization.name }}</strong> are using Sentry to catch errors, improve latency issues, and keep their (and maybe your) code from catching fire. 🔥</p>
Copy link

Choose a reason for hiding this comment

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

[BestPractice]

Template variable filtering inconsistency: The original code used {{ organization.name|sanitize_periods }} but the new code uses {{ organization.name }} without the filter. This could introduce security issues or rendering problems if organization names contain periods that need sanitization.

Suggested change
<p>Your teammates at <strong>{{ organization.name }}</strong> are using Sentry to catch errors, improve latency issues, and keep their (and maybe your) code from catching fire. 🔥</p>
<p>Your teammates at <strong>{{ organization.name|sanitize_periods }}</strong> are using Sentry to catch errors, improve latency issues, and keep their (and maybe your) code from catching fire. 🔥</p>

⚡ Committable suggestion

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Context for Agents 
[**BestPractice**]

Template variable filtering inconsistency: The original code used `{{ organization.name|sanitize_periods }}` but the new code uses `{{ organization.name }}` without the filter. This could introduce security issues or rendering problems if organization names contain periods that need sanitization.

```suggestion
    <p>Your teammates at <strong>{{ organization.name|sanitize_periods }}</strong> are using Sentry to catch errors, improve latency issues, and keep their (and maybe your) code from catching fire. 🔥</p>
```

⚡ **Committable suggestion**

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

File: src/sentry/templates/sentry/emails/member-invite.html
Line: 8


<p><a href="{{ url }}" class="btn">Join your team</a></p>
<p>Looks like they need your help. You've been invited to join them:</p>

<p>Check out the <a href="https://sentry.io/">Sentry website</a> if you'd like to learn more before diving in.</p>

<div class="notice">Do <strong>not</strong> share this invite email or link. This invite is unique to you. Sharing may result in unauthorized access.</div>
<p>👉 <a href="{{ url }}" class="btn">Accept your invite</a></p>

<div class="notice">Heads up: this invite link is just for you. Don't share it unless you want random strangers fixing your bugs.</div>
Copy link

Choose a reason for hiding this comment

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

[NitPick]

Inconsistent indentation: Line 15 has extra leading spaces (5 spaces instead of 4) compared to other elements at the same nesting level.

Suggested change
<div class="notice">Heads up: this invite link is just for you. Don't share it unless you want random strangers fixing your bugs.</div>
<div class="notice">Heads up: this invite link is just for you. Don't share it unless you want random strangers fixing your bugs.</div>

⚡ Committable suggestion

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Context for Agents 
[**NitPick**]

Inconsistent indentation: Line 15 has extra leading spaces (5 spaces instead of 4) compared to other elements at the same nesting level.

```suggestion
    <div class="notice">Heads up: this invite link is just for you. Don't share it unless you want random strangers fixing your bugs.</div>
```

⚡ **Committable suggestion**

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

File: src/sentry/templates/sentry/emails/member-invite.html
Line: 15


<p>Oh, and meet Seer, your new AI debugging agent. It'll automatically pinpoint root causes, suggest fixes, and even predict bugs before they land in prod.</p>

<p>
Welcome to the club.<br>
- The Sentry Team
</p>
{% endblock %}

{% block footer %}
Expand Down
15 changes: 9 additions & 6 deletions src/sentry/templates/sentry/emails/member-invite.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
You've been invited to Sentry
Hey there,

Your teammates at {{ organization.name }} are using Sentry to track and debug software errors.
Your teammates at {{ organization.name }} are using Sentry to catch errors, crush latency issues, and keep their (and maybe your) code from catching fire.

Join your team by visiting the following url:
Looks like they need your help. You've been invited to join them:

{{ url }}
{{ url }}

Do not share this invite email or link. This invite is unique to you. Sharing may result in unauthorized access.
Heads up: this invite link is just for you. Don’t share it unless you want random strangers fixing your bugs.

Check out the Sentry website (https://sentry.io) if you'd like to learn more before diving in.
Oh, and don't forget to try Seer, your new AI debugging agent. It’ll automatically pinpoint root causes, suggest fixes, and even predict bugs before they land in prod.

Welcome to the club.
- The Sentry Team