Skip to content

Conversation

@seer-by-sentry
Copy link
Contributor

@seer-by-sentry seer-by-sentry bot commented Oct 3, 2025

Fixes BLT-JAVASCRIPT-40-DH. The issue was that: Unauthenticated users cause JavaScript to call addEventListener on null because notification elements are conditionally rendered only for authenticated users.

  • Added a check to ensure notification elements exist before attempting to fetch notifications.
  • This prevents errors for unauthenticated users who do not have these elements on the page.

This fix was generated by Seer in Sentry, triggered by DB. 👁️ Run ID: 1755520

Not quite right? Click here to continue debugging with Seer.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 3, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@DonnieBLT DonnieBLT marked this pull request as ready for review October 3, 2025 16:05
@DonnieBLT DonnieBLT merged commit 45dffb9 into main Oct 3, 2025
11 of 13 checks passed
@DonnieBLT DonnieBLT deleted the seer/fix-notification-errors branch October 3, 2025 16:05
@github-project-automation github-project-automation bot moved this from Backlog to Done in 📌 OWASP BLT Project Board Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants