Skip to content

LIMS-2152: Remove notifications on load of beamline status page#1061

Merged
ndg63276 merged 1 commit into
pre-release/2026-R2.5from
fix/LIMS-2152/remove-notifications-on-bl-status
May 18, 2026
Merged

LIMS-2152: Remove notifications on load of beamline status page#1061
ndg63276 merged 1 commit into
pre-release/2026-R2.5from
fix/LIMS-2152/remove-notifications-on-bl-status

Conversation

@ndg63276
Copy link
Copy Markdown
Collaborator

JIRA ticket: LIMS-2152

Summary:

If you go to a visit and click "Beamline Status" to take you to eg /status/bl/i04, 2 or 3 notifications appear to say "Action Successful". We should silence these.

Changes:

  • Change order of arguments to the store.js functions, so the success function can be overwritten
  • Pass in a dummy function with no notification instead

To test:

  • Go to any visit, click "Beamline Status", check no notifications appear.

@ndg63276 ndg63276 changed the base branch from master to pre-release/2026-R2.5 May 18, 2026 16:14
@ndg63276 ndg63276 merged commit 614b7d3 into pre-release/2026-R2.5 May 18, 2026
2 checks passed
@ndg63276 ndg63276 deleted the fix/LIMS-2152/remove-notifications-on-bl-status branch May 18, 2026 16:16
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