SF-3917 Add a link to Low Draft Quality banner - #4104
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4104 +/- ##
=======================================
Coverage 81.00% 81.00%
=======================================
Files 664 664
Lines 42328 42331 +3
Branches 6988 6965 -23
=======================================
+ Hits 34286 34292 +6
- Misses 6876 6890 +14
+ Partials 1166 1149 -17 ☔ View full report in Codecov by Harness. |
RaymondLuong3
left a comment
There was a problem hiding this comment.
Waiting on the understanding-draft-quality help page to be made available before merging.
@RaymondLuong3 reviewed 9 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on pmachapman).
|
Looking at the diff makes me think this really should be a reusable component so we don't have strings and logic repeated several places. |
f57bc8a to
14e5fa7
Compare
@Nateowami Done - it was a bit more work than I expected, but I think it is simpler now. |
|
📸 Screenshot diff deployed! (3 changes) View the visual diff at: https://pr-4104--sf-screenshot-diffs.netlify.app |
14e5fa7 to
5571524
Compare
5571524 to
6c85416
Compare
6c85416 to
fffceb6
Compare
|
@RaymondLuong3 This is ready for re-review. Also the URL to the help site will need to be updated before we merge. But the help site article is live. |
RaymondLuong3
left a comment
There was a problem hiding this comment.
@RaymondLuong3 reviewed 15 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on pmachapman).
fffceb6 to
f6d1d75
Compare
See also:
This change is