Skip to content

escalation notifications#123

Open
ckulhanjian wants to merge 1 commit intoramfam101:developfrom
ckulhanjian:develop
Open

escalation notifications#123
ckulhanjian wants to merge 1 commit intoramfam101:developfrom
ckulhanjian:develop

Conversation

@ckulhanjian
Copy link
Copy Markdown

Describe your changes

I implemented escalation notifications as a configuration option when monitoring a site. Updated frontend to display the options and stored values in backend and database.

Write your issue number after "Fixes "

Fixes #123

Please ensure all items are checked off before requesting a review. "Checked off" means you need to add an "x" character between brackets so they turn into checkmarks.

  • (Do not skip this or your PR will be closed) I deployed the application locally.
  • (Do not skip this or your PR will be closed) I have performed a self-review and testing of my code.
  • I have included the issue # in the PR.
  • I have added i18n support to visible strings (instead of <div>Add</div>, use):
const { t } = useTranslation();
<div>{t('add')}</div>
  • I have not included any files that are not related to my pull request, including package-lock and package-json if dependencies have not changed
  • I didn't use any hardcoded values (otherwise it will not scale, and will make it difficult to maintain consistency across the application).
  • I made sure font sizes, color choices etc are all referenced from the theme. I don't have any hardcoded dimensions.
  • My PR is granular and targeted to one specific feature.
  • I ran npm run format in server and client directories, which automatically formats your code.
  • I took a screenshot or a video and attached to this PR if there is a UI change.

@smithareli smithareli mentioned this pull request Apr 8, 2026
10 tasks
@Aulakh1881 Aulakh1881 mentioned this pull request Apr 8, 2026
10 tasks
@Puckett33 Puckett33 mentioned this pull request Apr 8, 2026
10 tasks
@RyanTruonghuynh RyanTruonghuynh mentioned this pull request Apr 8, 2026
10 tasks
@amaanm12 amaanm12 mentioned this pull request Apr 9, 2026
10 tasks
@jwild2027 jwild2027 mentioned this pull request Apr 9, 2026
10 tasks
@AbbyBuchholz AbbyBuchholz mentioned this pull request Apr 9, 2026
10 tasks
@MsMaddie05 MsMaddie05 mentioned this pull request Apr 9, 2026
@lucasfonte21 lucasfonte21 mentioned this pull request Apr 9, 2026
8 tasks
@AstroProgram AstroProgram mentioned this pull request Apr 9, 2026
10 tasks
@Speedtrain49 Speedtrain49 mentioned this pull request Apr 9, 2026
@izzjuliana izzjuliana mentioned this pull request Apr 9, 2026
10 tasks
@vnkaralkar vnkaralkar mentioned this pull request Apr 9, 2026
10 tasks
@siddhantpallod siddhantpallod mentioned this pull request Apr 10, 2026
10 tasks
@alextarassiouk alextarassiouk mentioned this pull request Apr 10, 2026
10 tasks
@zsuero zsuero mentioned this pull request Apr 10, 2026
10 tasks
@Faybioo Faybioo mentioned this pull request Apr 11, 2026
10 tasks
@KolyCode KolyCode mentioned this pull request Apr 11, 2026
14 tasks
@annagracehaukoos annagracehaukoos mentioned this pull request Apr 12, 2026
10 tasks
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.

1 participant