X Tutup
Skip to content

Fix application id used for sharing settings translation#37846

Merged
micbar merged 1 commit intomasterfrom
bugfix/e4168
Aug 26, 2020
Merged

Fix application id used for sharing settings translation#37846
micbar merged 1 commit intomasterfrom
bugfix/e4168

Conversation

@VicDeo
Copy link
Member

@VicDeo VicDeo commented Aug 25, 2020

Description

'lib' was passed as the app name into the sharing settings panel instead of 'settings'

Related Issue

Screenshots (if appropriate):

section

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@VicDeo VicDeo added this to the development milestone Aug 25, 2020
@VicDeo VicDeo self-assigned this Aug 25, 2020
@VicDeo VicDeo changed the title Fix application id used for settings translation Fix application id used for sharing settings translation Aug 25, 2020
@VicDeo VicDeo force-pushed the bugfix/e4168 branch 2 times, most recently from 580cb15 to 39fc0f5 Compare August 25, 2020 11:18
@owncloud owncloud deleted a comment from update-docs bot Aug 25, 2020
@codecov
Copy link

codecov bot commented Aug 25, 2020

Codecov Report

Merging #37846 into master will decrease coverage by 0.00%.
The diff coverage is 90.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #37846      +/-   ##
============================================
- Coverage     64.75%   64.74%   -0.01%     
  Complexity    19396    19396              
============================================
  Files          1285     1285              
  Lines         75762    75762              
  Branches       1333     1333              
============================================
- Hits          49057    49055       -2     
- Misses        26313    26315       +2     
  Partials        392      392              
Flag Coverage Δ Complexity Δ
#javascript 54.03% <ø> (ø) 0.00 <ø> (ø)
#phpunit 65.93% <90.00%> (-0.01%) 19396.00 <1.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
settings/Panels/Admin/FileSharing.php 98.66% <88.88%> (ø) 7.00 <1.00> (ø)
lib/private/Settings/SettingsManager.php 72.95% <100.00%> (ø) 44.00 <0.00> (ø)
settings/templates/language.php 80.00% <0.00%> (-20.00%) 0.00% <0.00%> (ø%)
lib/private/Helper/LocaleHelper.php 97.36% <0.00%> (-2.64%) 15.00% <0.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 a992db8...aa9c078. Read the comment docs.

@phil-davis
Copy link
Contributor

codecov/patch is not going to come - that is failing to come nearly all the time these days.

Someone needs to decide if the codecov/project result is acceptable, and merge...

@micbar

@micbar micbar merged commit b56e821 into master Aug 26, 2020
@delete-merged-branch delete-merged-branch bot deleted the bugfix/e4168 branch August 26, 2020 13:23
@jnweiger
Copy link
Contributor

##Retested with 10.6.0 RC1

  • "Sprache, die für öffentliche Mail-Benachrichtigungen für geteilte Daten benutzt wird" -> this setting no longer exists. OK
  • "[ ] Erstellen [ ] Ändern [ ] Löschen [ ] Teilen" -> OK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup