Skip to content
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

feat(pam/nativemodel): Do not render qrcode but show uri only in pkexec #405

Merged
merged 3 commits into from
Jul 5, 2024

Conversation

3v1n0
Copy link
Collaborator

@3v1n0 3v1n0 commented Jul 1, 2024

Do not include qrcode in polkit UI

Schermata del 2024-07-01 18-20-41

UDENG-3422

@3v1n0 3v1n0 requested a review from a team as a code owner July 1, 2024 16:49
@3v1n0 3v1n0 force-pushed the qrcode-show-uri-only-pkexec branch from d6d655e to 42ea6a3 Compare July 1, 2024 17:10
@codecov-commenter
Copy link

codecov-commenter commented Jul 2, 2024

Codecov Report

Attention: Patch coverage is 91.17647% with 3 lines in your changes missing coverage. Please review.

Project coverage is 84.47%. Comparing base (8e73e09) to head (cbe6f3b).

Files Patch % Lines
pam/internal/adapter/nativemodel.go 90.00% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #405      +/-   ##
==========================================
+ Coverage   84.38%   84.47%   +0.09%     
==========================================
  Files          77       77              
  Lines        6713     6733      +20     
  Branches       75       75              
==========================================
+ Hits         5665     5688      +23     
+ Misses        734      731       -3     
  Partials      314      314              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@denisonbarbosa denisonbarbosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't understand a single word of the picture that you added on the PR description, but at least we don't have a qrcode there 😂

Jokes apart, looks good to me! Well done.

@3v1n0 3v1n0 requested a review from didrocks July 3, 2024 06:17
@3v1n0 3v1n0 force-pushed the qrcode-show-uri-only-pkexec branch from 6c258ea to 1b3a0af Compare July 4, 2024 14:09
@3v1n0 3v1n0 force-pushed the qrcode-show-uri-only-pkexec branch from 1b3a0af to cbe6f3b Compare July 4, 2024 22:25
@3v1n0
Copy link
Collaborator Author

3v1n0 commented Jul 4, 2024

Rebased updating the golden file to respect the changes in #408

@3v1n0 3v1n0 requested a review from denisonbarbosa July 4, 2024 22:26
@3v1n0 3v1n0 merged commit 25eafef into ubuntu:main Jul 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants