Display fallback message if not refund reason provided#5887
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
There was a problem hiding this comment.
Pull Request Overview
This PR restores a fallback message for refunds that lack both refund reason type and refund reason note. When neither value is provided, the refund table will now display "manual refund" as a fallback instead of showing empty content.
- Adds a fallback display message for refunds missing both reason type and reason note
- Imports refund grid messages for the fallback text
- Fixes a failing e2e test that expects this fallback text to be present
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5887 +/- ##
=========================================
Coverage 38.64% 38.65%
=========================================
Files 2742 2742
Lines 43228 43231 +3
Branches 9449 9803 +354
=========================================
+ Hits 16707 16710 +3
+ Misses 26491 25209 -1282
- Partials 30 1312 +1282 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
In the new refund reasons feature instead of "manual refund" we display refund type and/or arbitrary refund reason.
In case of missing both of them (not provided by users), refunds table displays empty message.
This PR restores "manual refund" message as a fallback value. It also fixes failing e2e test that asserts this text on the screen