Derrick catherine/refund cancellation fix updates

[xiec1377]

Notion ticket link

(no notion ticket)

1. visual differences:
   (a) include “Camp Cancellation” in heading
   (b) textStyle for subheading is different
   (c) no total refund at the bottom

2. functional differences
   (a) doesn’t fail gracefully on bad refund code (FE tries to map refunds which doesn’t exist and crashes, we probably want this to redirect to main page with a toast about the failure)
   (b) EDLP should be hidden if it’s price is $0
   (c) having a loading spinner on this would be a “nice to have”, search the code base for “spinner” for examples

3. code style:
   (a) we’ll want to use existing textStyles rather than hardcoding new font sizes (see my draft PR for example)
   (b) lots of lint warnings about types and unused variables, we want to clean those up before they hit the main branch

Implementation description

• For visual differences, added textStyle and button sizing for dynamic page sizing
• For functional differences,
  • loading state first set to true which displays spinner; in useEffect() function, if valid API call, set validCode to true; else set validCode to false, display toast error and return to home page; finally set loading to false
  • if getEDLPTotalCharge() = $0, then hide EDLP section

Steps to test

1. Go to link to test for the above requirements

What should reviewers focus on?

• Test functional bugs
• Check if visually looks the same to the Figma for all sm, md, lg sizing

Checklist

• My PR name is descriptive and in imperative tense
• My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
• I have run the appropriate linter(s)
• I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

[github-actions]

Visit the preview URL for this PR (updated for commit d67e4d3):

https://focus-on-nature--pr289-derrick-catherine-re-zyvzh10f.web.app

_{(expires Sat, 05 Aug 2023 21:51:54 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 0aeb3dd1cc2aa82265739c366a8befd0d956fd1f}

[williamtran10]

Overall, good job addressing the big list of changes. Preferably in the future if you make linting changes to files that are out of the scope of the ticket, include them all in a separate PR (thanks for fixing them though).

[williamtran10]

Also still a couple small lint changes to make

[derrick-yyang]

Also still a couple small lint changes to make

I think the last lint warnings should be left. One of them is related to key which is required from using the .maps function.
The other one is unrelated to our changes, not sure if we actually want it or not

[williamtran10]

Only a couple of small changes left. Also in the future, it's a good idea to let the reviewer mark conversations as resolved so that it's easier for them to check any changes that were requested.

[williamtran10]

Also double check the most recent commit 637f5c5, it might have reverted some of your fixes

[williamtran10]

Good work and good job getting through this slog of a ticket. Feel free to merge