Change the wording in the additional tab to not confuse users

[ChrisEdS]

Description

This PR changes two strings that caused confusion in the additional tab.

Related Issue

• Fixes https://github.com/owncloud/enterprise/issues/2500

Motivation and Context

Confusion should be avoided.

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

[phil-davis]

and needs a changelog
has been done

[codecov]

Codecov Report

Merging #36775 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #36775   +/-   ##
=========================================
  Coverage     64.92%   64.92%           
  Complexity    19151    19151           
=========================================
  Files          1267     1267           
  Lines         74902    74902           
  Branches       1331     1331           
=========================================
  Hits          48633    48633           
  Misses        25877    25877           
  Partials        392      392           

Flag	Coverage Δ	Complexity Δ
#javascript	54.14% <ø> (ø)	0.00 <ø> (ø)
#phpunit	66.13% <ø> (ø)	19151.00 <ø> (ø)

Impacted Files	Coverage Δ	Complexity Δ
settings/templates/settingsPage.php	0.00% <ø> (ø)	0.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 5ee4168...12e8fea. Read the comment docs.

[phil-davis]

I added 1 word and added a changelog.
Tested manually - the text looks fine to me on the webUI.
@pmaier1 please review

[pmaier1]

Text is fine. Thanks for taking care everybody 👌

[micbar]

Needs to be decided when to merge.

Currently it would go into 10.4 but we didn't update the translations.

[phil-davis]

Note: PR #36776 changed the word "Error" to "Other" on that page. So that "fixed" it for 10.4.

I rebased this just now.

After 10.4 is released, someone can decide if they would like to have the words in this PR, and merge it.

[micbar]

@phil-davis thanks. I think everybody agreed on the solution.

[mmattel]

Ping ?

[phil-davis]

This is waiting until after 10.4.0 is released. 10.4.0 changed the word "Error" to "Other", which at least gets rid of the misleading word "Error".

If we had realised how long it would be until we got 10.4.0 released, then we could have merged this and the new translations would have had plenty of time to be done. Anyway, too late for that now.

[phil-davis]

I rebased this and adjusted the words of the changelog entry.
With 10.4.0 release "real soon now", we could consider merging this next week for 10.4.1

[phil-davis]

This missed out on 10.4.1
If we want it, then we could merge it this week (e.g. straight after 10.4.1 branch gets merged back...). And that will then give time to do translations before the next release.

[phil-davis]

Nobody merged this.
@micbar if we merge this now, will necessary translations have time to happen and get back from Transifex to core master before an official 10.5.0 branch is created?