Skip to content

Project Alerts

Emily Jensen requested to merge 5-project-alerts into master

Project Alerts

image

This merge request displays and updates a users' current list of recipients. Images of most of the states are displayed below.

  • This is missing Adding a new alert and a new alert recipient (I commented out "Create New Alert" and "Add Alert Recipient Buttons, in case you want to merge)
  • I tried setting up state using several variables for loading and errors. What do you think of that?
  • I created separate reducer functions for each action
  • This request also includes the addition of two SVG's (one for Slack, which we will need down the road, and one for a webhook, since angular material does not have a webhook option)
  • I made changes to app-loading-button to cover more display options that we needed for this component
  • I think we could do some nicer error handling, but we are checking for errors with every http request.
  • The snackbar pops up with each success response (except for initial load because that would be weird)

Initial Load

image

Initial Load Error

image

Empty State

image

Loading States

  • These wouldn't typically occur at the same time.

image

Error States

  • This is for display purposes. Typically they wouldn't occur at the same time.

image

No Alert Recipients

image

Related to #5 (closed)

Edited by Emily Jensen

Merge request reports