chore(intercom): replace zendesk with new help center#114609
chore(intercom): replace zendesk with new help center#114609sentaur-athena merged 5 commits intomasterfrom
Conversation
github-actions
Bot
added
Scope: Frontend
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
| </p> | ||
| <p><a href="{{ view_monitors_link }}" class="btn">View Monitors</a></p> | ||
| <p> | ||
| Still facing issues? <a href="https://sentry.zendesk.com/hc/en-us/requests/new">Reach out to our support team</a>. | ||
| Still facing issues? <a href="https://www.sentry.help/">Reach out to our support team</a>. | ||
| </p> | ||
| {% endblock %} | ||
|
|
There was a problem hiding this comment.
Bug: The support link in the broken-monitors.html email template incorrectly points to the help center homepage instead of the new request form.
Severity: LOW
Suggested Fix
In broken-monitors.html, change the href for the support link from https://www.sentry.help/ to https://www.sentry.help/requests/new to ensure users are directed to the correct support request form.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: src/sentry/templates/sentry/emails/crons/broken-monitors.html#L30-L36
Potential issue: The 'Reach out to our support team' link in the `broken-monitors.html`
email template was changed to point to the general help center homepage
(`https://www.sentry.help/`) instead of the support request form. This is inconsistent
with other similar links in the codebase, such as in `broken-monitors.txt` and
`muted-monitors.html`, which correctly use `https://www.sentry.help/requests/new`. This
change creates a functional regression, as users are no longer taken directly to the
support form, adding an extra step to the process of seeking help.
Did we get this right? 👍 / 👎 to inform future reviews.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit bce7374. Configure here.
|
|
||
| {% block footer %} | ||
| <a href="https://sentry.zendesk.com/hc/en-us/">Help Center</a> | ||
| <a href="https://sentry.help/">Help Center</a> |
There was a problem hiding this comment.
Missing www. prefix in help center URL
Medium Severity
The member invite email uses https://sentry.help/ while every other URL in this PR consistently uses https://www.sentry.help/. This could point to a different (or non-existent) domain, resulting in a broken Help Center link for new members receiving their invitation email.
Reviewed by Cursor Bugbot for commit bce7374. Configure here.
There was a problem hiding this comment.
Seems legit? If nothing else might be nice to be consistent.
| <p><a href="{{ view_monitors_link }}" class="btn">View Monitors</a></p> | ||
| <p> | ||
| Still facing issues? <a href="https://sentry.zendesk.com/hc/en-us/requests/new">Reach out to our support team</a>. | ||
| Still facing issues? <a href="https://www.sentry.help/">Reach out to our support team</a>. |
There was a problem hiding this comment.
Broken-monitors HTML drops /requests/new support path
Medium Severity
The broken-monitors HTML email links to https://www.sentry.help/ (the help center homepage) instead of https://www.sentry.help/requests/new (the support request form). The corresponding .txt template correctly uses /requests/new, as do both muted-monitors.html and muted-monitors.txt. The link text says "Reach out to our support team," so the intent is to link to the request form, not the homepage.
Reviewed by Cursor Bugbot for commit bce7374. Configure here.
saponifi3d
left a comment
saponifi3d
left a comment
There was a problem hiding this comment.
Lgtm, just the (probably nit) to add www on the one url.
| {t('Take a tour')} | ||
| </Button> | ||
| <ExternalLink href="https://sentry.zendesk.com/hc/en-us/articles/48882501173915"> | ||
| <ExternalLink href="https://www.sentry.help/articles/48882501173915"> |
There was a problem hiding this comment.
do we need a redirect from sentry.help to www.sentry.help? wonder if users might navigate to the wrong on since browsers hide www. by default now.
There was a problem hiding this comment.
It does work right now but I got dns resolution earlier when I didn't enter www.
There was a problem hiding this comment.
ah yup works for me now too 👍🏼
Co-authored-by: George K. <62101637+t-h-man@users.noreply.github.com>
4 participants
No description provided.