US-EPA-CAMD / easey-ui

Project Management repo for EPA Clean Air Markets Division (CAMD) Business Suite of applications
MIT License
0 stars 0 forks source link

Add error handling for Content Manager Human/Input Error #2663

Closed JanellC closed 6 months ago

JanellC commented 2 years ago

Definition of Ready

User Story

As the EPA I want the EASEY content management system to have error handling in place So that I don't break CAMPD when making updates to a JSON file

Acceptance Criteria

Given that a user has made an inaccurate update to a JSON file in in the EASEY content management system When the CAMPD functionality is displayed Then the system displays a USWDS alert with the Content Manager Human/Input Error (JSON file error): (Refer to the CAMPD downtime document )

Development Notes

Does this Story require 508 Testing?

Context

(User journeys, design mockups, architectural diagrams, business rules, data element definitions)

https://designsystem.digital.gov/components/site-alert/

See spike #2800

https://usepa.sharepoint.com/:x:/r/sites/CAMDCVPTeam/_layouts/15/Doc.aspx?sourcedoc=%7B33677C82-CD21-4DE2-B3EA-7FCE5907452A%7D&file=CAMPD%20Downtime%20Errors%20-%20EPA%20CAMD%20-%20v1.0.xlsx&action=default&mobileredirect=true

Testing

Test Case: Tester:

Definition of Done

JanellC commented 2 years ago

Per backlog refinement on 6/1, this functionality will not be included in the 1.1 release

hamezene commented 9 months ago

need more research

ergjustin commented 8 months ago

IN DEV ENVIRONMENT When I test API error banner, I found one issue that is in contact page email API shows 404 error []() Private Zenhub Image

ergjustin commented 8 months ago

PR

https://github.com/US-EPA-CAMD/easey-campd-ui/pull/611

Changes

Added Input error handling by using yup

Step to test

  1. easey-campd-ui connect to easey-content locally using REACT_APP_EASEY_CONTENT_API
  2. Change the value of campd json file and test
shahbazkhan1999 commented 8 months ago

@bchofor start to test this one need any help coordinate @ergjustin

ergjustin commented 8 months ago

Developer test in dev env I made an input error in easey-content /data/home.index.json (removing name) then throws error without app crushing and display contentManager error, contentManager Image

tlageman-epa commented 8 months ago

looks good in staging