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

do not lower case search provider names on web ui #25312

Merged
merged 2 commits into from
Jan 27, 2021
Merged

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Jan 25, 2021

fixes #25282

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz
Copy link
Member Author

blizzz commented Jan 25, 2021

/backport to stable20

@nickvergessen
Copy link
Member

you didn't build production, right?

@blizzz
Copy link
Member Author

blizzz commented Jan 25, 2021

you didn't build production, right?

oops

@blizzz
Copy link
Member Author

blizzz commented Jan 25, 2021

/compile amend /core/

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

Fine by me but I think this was there for a reason

@rullzer rullzer merged commit 75300cb into master Jan 27, 2021
@rullzer rullzer deleted the fix/25282/delowercase branch January 27, 2021 13:51
@backportbot-nextcloud
Copy link

The backport to stable20 failed. Please do this backport manually.

@rullzer rullzer mentioned this pull request Jan 27, 2021
19 tasks
@nickvergessen
Copy link
Member

Fine by me but I think this was there for a reason

It was there to be correct in English. The current way this string is build is just not conforming with our translation system. But due to the lack of other ideas... (So it actually comes from the rule that in English we uppercase the first word always, because 'Talk', 'Files', and the other search provider names should actually not be uppercase to begin with.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Search provider names in unified search hint do not respect localized language rules
5 participants