Skip to content

refactor: Refactor alert and incident property handling. #3941

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

VladimirFilonov
Copy link
Contributor

Introduce cached properties for alerts and incidents to ensure data enrichment on first access. Refactor related code for better encapsulation and consistency, replacing direct attribute access with structured methods.

Closes #

📑 Description

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

Copy link

vercel bot commented Mar 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
keep ⬜️ Ignored (Inspect) Visit Preview Mar 10, 2025 7:31am

Copy link
Contributor

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

Introduce cached properties for `alerts` and `incidents` to ensure data enrichment on first access. Refactor related code for better encapsulation and consistency, replacing direct attribute access with structured methods.
@VladimirFilonov VladimirFilonov force-pushed the refactor/encapsulate-_alerts-n-_incidents branch from 51e85f6 to 9daaf60 Compare March 10, 2025 07:31
@talboren
Copy link
Member

@VladimirFilonov closing this draft for now as it's hanging for a long time. let's re-open it when it's ready

@talboren talboren closed this Apr 14, 2025
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.

2 participants