Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

For #18729 - Use consistent string naming for generic strings #18730

Merged
merged 1 commit into from
Mar 31, 2021

Conversation

gabrielluong
Copy link
Member

Fixes #18729

We don't use R.string.edit anywhere except in a nav_graph label and snackbar test. https://github.com/mozilla-mobile/fenix/search?q=R.string.edit. So, I am moving that up to where we have our generic strings. I think this is sufficiently generic that we don't need to create a new edit_label string as done in #18555 (comment).

Pull Request checklist

  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features. In addition, it includes a screenshot of a successful accessibility scan to ensure no new defects are added to the product.

To download an APK when reviewing a PR:

  1. click on Show All Checks,
  2. click Details next to "Taskcluster (pull_request)" after it appears and then finishes with a green checkmark,
  3. click on the "Fenix - assemble" task, then click "Run Artifacts".
  4. the APK links should be on the left side of the screen, named for each CPU architecture

@gabrielluong gabrielluong requested review from a team as code owners March 31, 2021 15:47
@gabrielluong gabrielluong added the needs:review PRs that need to be reviewed label Mar 31, 2021
@codecov-io
Copy link

Codecov Report

Merging #18730 (fd37440) into master (2687092) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #18730      +/-   ##
============================================
- Coverage     33.45%   33.44%   -0.02%     
- Complexity     1435     1436       +1     
============================================
  Files           499      499              
  Lines         20504    20540      +36     
  Branches       3094     3100       +6     
============================================
+ Hits           6860     6869       +9     
- Misses        12796    12824      +28     
+ Partials        848      847       -1     
Impacted Files Coverage Δ Complexity Δ
...lla/fenix/components/toolbar/DefaultToolbarMenu.kt 3.30% <0.00%> (ø) 2.00 <0.00> (ø)
...a/org/mozilla/gecko/search/SearchWidgetProvider.kt 24.00% <0.00%> (-6.77%) 1.00% <0.00%> (ø%)
...java/org/mozilla/fenix/perf/PerformanceInflater.kt 95.00% <0.00%> (+15.00%) 5.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2687092...fd37440. Read the comment docs.

@mergify
Copy link
Contributor

mergify bot commented Mar 31, 2021

This pull request has conflicts when rebasing. Could you fix it @gabrielluong? 🙏

@jonalmeida jonalmeida added pr:needs-landing PRs that are ready to land [Will be merged by Mergify] and removed needs:review PRs that need to be reviewed labels Mar 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr:needs-landing PRs that are ready to land [Will be merged by Mergify]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use consistent string naming for generic strings
3 participants