-
Notifications
You must be signed in to change notification settings - Fork 48
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
Luis add email notification to warning tracker #2752
Merged
one-community
merged 21 commits into
development
from
luis-add-email-notification-to-warning-tracker
Dec 18, 2024
Merged
Changes from 19 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
82690d1
restored tracker modal
luisarevalo21 93b9181
WIP: da9c70f33 saving work
luisarevalo21 f3d7d89
added files for frontend add/remove warnings
luisarevalo21 5159481
resolved warning crashing by adding correct method calls for handleTo…
luisarevalo21 e2e07c9
completed frontend
luisarevalo21 63dfe5c
resolved lint issue
luisarevalo21 86c59c0
resolved duplicate bug and editing new warning
luisarevalo21 3cef7c1
removed old function
luisarevalo21 f4f4f63
resolved merge conflict
luisarevalo21 0348bc7
adjusted styling for new warning form
luisarevalo21 e6d7482
attempting to send an email to myself
luisarevalo21 1de5aaf
removed alert and added monitordata to functions
luisarevalo21 79d71a0
added sending email functionality and hid the issue warning, and adde…
luisarevalo21 b8e0276
added styling warning text for more than 3 warnings"
luisarevalo21 773ec45
adjusted popup modal text
luisarevalo21 69160c2
adjusted modal text when issuing a warning
luisarevalo21 5b1db9c
removed test code
luisarevalo21 1e6eb36
removed more test code
luisarevalo21 df40b49
removed logs
luisarevalo21 65fa82a
merged dev into branch
luisarevalo21 ab30908
removed comments in url file
luisarevalo21 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Errors should never get a
200
, this also requires backend changes.