-
-
Notifications
You must be signed in to change notification settings - Fork 4.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
Bring back the description for app passwords #5265
Conversation
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @schiessle, @ChristophWurst and @LukasReschke to be potential reviewers. |
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.
LGTM! thanks for taking over! :)
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.
Tested and works 👍
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Codecov Report
@@ Coverage Diff @@
## master #5265 +/- ##
=========================================
Coverage 54.17% 54.17%
Complexity 22302 22302
=========================================
Files 1380 1380
Lines 85383 85383
Branches 1321 1321
=========================================
Hits 46259 46259
Misses 39124 39124
|
Please backport. |
It's already in the PR of the initial backport which caused this regression: |
Nice, thanks! :) |
Follow up to #5166