Uh oh!
There was an error while loading. Please reload this page.
Updates syncing metadata fields on syncing modal - #4016
Conversation
bjester
left a comment
There was a problem hiding this comment.
This looks great! It'll need some testing, but the only change that I can see is the usage of waitForCopying is unnecessary because of your watch approach. Sorry I threw you off with mentioning that.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Hi @bjester, the css has been applied as you recommended, with very minor tweaks. All seems to look good now. |
bjester
left a comment
There was a problem hiding this comment.
Good to go! some merge conflicts though
akolson
commented
Apr 14, 2023
All good now. Thanks Blaine! |






Summary
Description of the change(s) you made
This pr updates the syncing modal to look as shown below;

Manual verification steps performed
See #3986
Screenshots (if applicable)
Does this introduce any tech-debt items?
No
Reviewer guidance
How can a reviewer test these changes?
/en/channels/#/my-channelsAre there any risky areas that deserve extra testing?
References
Fixes#3986
Comments
Contributor's Checklist
PR process:
CHANGELOGlabel been added to this PR. Note: items with this label will be added to the CHANGELOG at a later timedocslabel has been added if this introduces a change that needs to be updated in the user docs?requirements.txtfiles also included in this PRStudio-specifc:
notranslateclass been added to elements that shouldn't be translated by Google Chrome's automatic translation feature (e.g. icons, user-generated text)pages,components, andlayoutsdirectories as described in the docsTesting:
Reviewer's Checklist
This section is for reviewers to fill out.
yarnandpip)