Skip to content
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

fix: incorrect key breaks page #354

Merged
merged 2 commits into from
Sep 13, 2024
Merged

fix: incorrect key breaks page #354

merged 2 commits into from
Sep 13, 2024

Conversation

jansenk
Copy link
Contributor

@jansenk jansenk commented Sep 12, 2024

https://2u-internal.atlassian.net/browse/AU-2187

If there was an error rendering a file, rather than showing an error message, we would unfortunately pass a message with the wrong key and so pass undefined to FormattedMessage (which would crash the page)

Now? Tada
Screenshot 2024-09-12 at 4 02 38 PM

@jansenk jansenk changed the title fix: incorrect key crashes page fix: incorrect key breaks page Sep 12, 2024
Copy link

codecov bot commented Sep 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (567a020) to head (d608dac).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #354   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          111       111           
  Lines         1082      1082           
  Branches       159       159           
=========================================
  Hits          1082      1082           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@KristinAoki KristinAoki merged commit 7003b61 into master Sep 13, 2024
7 checks passed
@KristinAoki KristinAoki deleted the jkantor/headingMessage branch September 13, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants