Skip to content

Conversation

@Ahalya-ni
Copy link
Collaborator

@Ahalya-ni Ahalya-ni commented Nov 18, 2025

Pull Request

🤨 Rationale

Task 3513145: PR2 | Remove Alarms Count reference

This PR removes all references to Alarms Count from the query type control, editors, query handlers, and related code.

Note: I've set the branch users/ahalya/feat/output-type-ui as the base for this branch to ensure this alarms count removal PR can only be merged after the output type is introduced. This provides a safe migration if any issues arise, we can refer the alarms count datasource as a backup.

👩‍💻 Implementation

  • Removed Alarms Count from Query Type enum.
  • Deleted the AlarmsCountEditor and AlarmsCountQueryHandler.
  • Cleaned up all other usages of alarms count throughout the datasource.

🧪 Testing

  • Updated and removed tests related to alarms count, including associated test files.
image

✅ Checklist

@Ahalya-ni Ahalya-ni changed the title refactor(alarms): Remove alarms count from query type refactor(alarms): Remove Alarms Count query type and related code Nov 18, 2025
@Ahalya-ni Ahalya-ni requested a review from manisha-ni November 18, 2025 06:23
Signed-off-by: Ahalya Radhakrishnan <[email protected]>
@Ahalya-ni Ahalya-ni force-pushed the users/ahalya/feat/remove-alarms-ref branch from f6139b5 to a8de716 Compare November 19, 2025 11:26
@Ahalya-ni Ahalya-ni marked this pull request as ready for review November 19, 2025 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants