[Resolve Errors] [Review Warnings] Exceed error warning limit alert & counts read off the response body

[shindigira]

closes #623
closes #624
closes #630
closes #704

Changes

• feat(Errors/Warnings): A field alert alert appears when the error/warnings limit is exceeded (i.e. is_truncated)
• feat(Errors/Warnings): Single Field, Multi Field and Register counts now come from the data response object
• enhancement(Download Validation Report): Added button loader
• enhancement(Field Level Alert - Download Validation Report Link): Now scrolls to the Download Validation Report button

How to Test

• Upload 1_000_000 errors
• Navigate to Errors and/or Warnings
• Ensure the counts properly match the data response object
• The Field Level Alert appears when necessary
• Test the Download Validation report link

Screenshot

[meissadia]

Not working for me. I am seeing zeroes in the UI for error counts when there are 400 logic errors shown in /submission/latest response, as well as errors shown in tables.

I think the Download Report function is working well. Downloaded report shows 200,727 entries - I'm not sure what source I could compare that against to determine if that is the correct number of triggered validations.

API response

UI

[shindigira]

@meissadia You have to yarn update to the latest filing changes. Justin just merged in the count changes today.

[meissadia]

@shindigira Sorry for that oversight and thanks for the clarification, I've updated my backend

• The numbers displayed in the UI seem to match what I'm seeing in the API. 👍🏾
• Download report looks good 👍🏾
• I do see one upload a new file link missing form the error Alert box
  • 

[shindigira]

@shindigira Sorry for that oversight and thanks for the clarification, I've updated my backend

• The numbers displayed in the UI seem to match what I'm seeing in the API. 👍🏾

• Download report looks good 👍🏾

• I do see one upload a new file link missing form the error Alert box

  • 

@meissadia The filing alert has two mockups in figma where one has the link and one does not; should confirm with Natalia which she wants. Also, since it is outside the scope of this current PR's aim, we should deal with it in a different PR.

[meissadia]

LGTM! 👍🏾

[natalia-fitzgerald]

@shindigira
Please see related feedback posted here: #578

[billhimmelsbach]

I don't think you had the chance to merge this great PR in before ya left on vacation, so going to do that now just to avoid some conflicts with related PRs.