Fix some string mappings for link new device errors#6675
Fix some string mappings for link new device errors#6675hughns wants to merge 5 commits intoelement-hq:developfrom
Conversation
|
Thank you for your contribution! Here are a few things to check in the PR to ensure it's reviewed as quickly as possible:
|
|
I think you need to record new screenshots:
That said, recording screenshots may be failing for forks at the moment. I'm trying to figure out if it's a permission issue in a fork of mine. That should be fixed once #6679 is merged. Ok, the fix is merged in |
bmarty
left a comment
There was a problem hiding this comment.
That sounds good, thanks!
Do not bother about the unused strings, I can handle that later.
Waiting for the screenshots to be updated before approving the changes.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #6675 +/- ##
===========================================
- Coverage 81.22% 81.21% -0.01%
===========================================
Files 2628 2628
Lines 73255 73255
Branches 9494 9494
===========================================
- Hits 59499 59494 -5
- Misses 10270 10277 +7
+ Partials 3486 3484 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@hughns I think we need to merge the |

Content
Some of the link new device error screens are currently using strings for the login flow (not the link new device flow).
This PR updates those screens to use the (previously unused)
link_new_device_*strings.Alternatively, the unused strings should be removed.
Motivation and context
Screenshots / GIFs
Tests
Tested devices
Checklist