Skip to content

Adds code to improve the access check before sending an alert, #AS-586#262

Merged
AltamashShaikh merged 2 commits into
5.x-devfrom
AS-586-check-access
Jun 24, 2026
Merged

Adds code to improve the access check before sending an alert, #AS-586#262
AltamashShaikh merged 2 commits into
5.x-devfrom
AS-586-check-access

Conversation

@AltamashShaikh

Copy link
Copy Markdown
Contributor

Description

Adds code to improve the access check before sending an alert

Issue No

#AS-586

Steps to Replicate the Issue

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [✔] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [✔] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

@lachiebol lachiebol left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread CHANGELOG.md Outdated
@AltamashShaikh AltamashShaikh requested a review from lachiebol June 23, 2026 01:41
@AltamashShaikh AltamashShaikh merged commit 104b690 into 5.x-dev Jun 24, 2026
9 checks passed
@AltamashShaikh AltamashShaikh deleted the AS-586-check-access branch June 24, 2026 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants