-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[PM-7349] Update snap description with new URL to help docs #8703
Conversation
I have a feeling we're using the wrong tag here. There's a |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8703 +/- ##
==========================================
+ Coverage 27.20% 27.62% +0.42%
==========================================
Files 2339 2356 +17
Lines 68263 68972 +709
Branches 12748 12924 +176
==========================================
+ Hits 18570 19055 +485
- Misses 48297 48488 +191
- Partials 1396 1429 +33 ☔ View full report in Codecov by Sentry. |
New Issues
|
@jlf0dev I've updated to have a short summary and your original description, with the inclusion of the new help URL as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks!
* Updated snap summary with new URL to help docs. * Updated to use summary and description. (cherry picked from commit e4ebf4a)
Type of change
Objective
In #8672 we added a description to the snap build to indicate that we need users to manually connect the
password-manager-service
. This turned out to be longer than the 128-character max on the description.We shortened it temporarily in #8687 but have since added help docs and this is adjusting the description to link to the docs.
Before you submit