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

postmarketos: use pretty_name for display labels #3324

Merged
merged 1 commit into from
May 11, 2024

Conversation

z3ntu
Copy link
Contributor

@z3ntu z3ntu commented Jun 7, 2023

BPO now has pretty_name fields for interfaces, releases (and devices). Use them and remove the hardcoded UI name mappings.

@codecov
Copy link

codecov bot commented Jun 7, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.99%. Comparing base (4c27ca2) to head (7fd5394).

❗ Current head 7fd5394 differs from pull request most recent head 04667fc. Consider uploading reports for the commit 04667fc to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3324      +/-   ##
==========================================
- Coverage   84.06%   83.99%   -0.08%     
==========================================
  Files          32       32              
  Lines        1086     1081       -5     
==========================================
- Hits          913      908       -5     
  Misses        173      173              

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

@amartinz amartinz requested a review from maciek134 June 8, 2023 07:23
@z3ntu
Copy link
Contributor Author

z3ntu commented Jul 6, 2023

Bump?

@z3ntu
Copy link
Contributor Author

z3ntu commented Dec 12, 2023

Review please :)

Flohack74
Flohack74 previously approved these changes Mar 10, 2024
@z3ntu z3ntu force-pushed the pmos-pretty_name branch from 5725027 to 7fd5394 Compare March 11, 2024 14:09
@z3ntu
Copy link
Contributor Author

z3ntu commented Mar 11, 2024

Ugh one of the security issues reported by CI is caused by progressive-downloader dependencies but that was pinned to that version with #3243

@z3ntu
Copy link
Contributor Author

z3ntu commented Mar 11, 2024

FTR updated the dependencies of progressive-downloader for the 1.x version here: https://gitlab.com/z3ntu/progressive-downloader-node/-/commits/1.x

@z3ntu
Copy link
Contributor Author

z3ntu commented Mar 14, 2024

Flohack74
Flohack74 previously approved these changes May 9, 2024
@z3ntu z3ntu dismissed Flohack74’s stale review May 9, 2024 20:30

The merge-base changed after approval.

BPO now has pretty_name fields for interfaces, releases (and devices).
Use them and remove the hardcoded UI name mappings.
@z3ntu z3ntu force-pushed the pmos-pretty_name branch from 8a2443f to 04667fc Compare May 9, 2024 20:50
@Flohack74 Flohack74 merged commit 5ed246b into ubports:master May 11, 2024
9 checks passed
@z3ntu z3ntu deleted the pmos-pretty_name branch May 12, 2024 07:39
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.

2 participants