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

[dev] Allow more locales to be used by FreeTube #2322

Merged

Conversation

ChunkyProgrammer
Copy link
Member


[dev] Allow more locales to be used by FreeTube

Pull Request Type
Please select what type of pull request this is:

  • Bugfix
  • Feature Implementation

Related issue
I noticed a lot of changes recently for languages that are not currently displayed in the FreeTube app

Description
This PR allows the following locales to be selected:
ca (català/catalan), es_AR, et (Eesti/Estonian), eu (euskera/basque), ko (한국인/ korean), lt (lietuvių/lithuanian)

Testing (for code that is not small enough to be easily understandable)
Has this pull request been tested?
Please describe shortly how you tested it and whether there are any ramifications remaining.
I tested by changing my locale to the languages that were added.

Desktop (please complete the following information):

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.17.0 RC

Additional context
Tested on dev branch with latest locale changes (Basque in particular got a lot of updates not in the RC branch so it won't appear to be translated as much on that branch)

@PrestonN PrestonN enabled auto-merge (squash) June 15, 2022 10:43
@ChunkyProgrammer ChunkyProgrammer added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 15, 2022
@PrestonN PrestonN enabled auto-merge (squash) June 15, 2022 10:51
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc dismissed their stale review June 19, 2022 18:37

forgot to check Basque/Euskera

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Jun 19, 2022
@ChunkyProgrammer
Copy link
Member Author

@efb4f5ff-1298-471a-8973-3d47447115dc are you having issues on this branch too?

@ChunkyProgrammer ChunkyProgrammer added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: changes requested labels Jun 19, 2022
Copy link
Collaborator

@PikachuEXE PikachuEXE left a comment

Choose a reason for hiding this comment

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

Is it possible to put those locales in more lines?
My brain hurts reading that long line...

@efb4f5ff-1298-471a-8973-3d47447115dc

Is it possible to put those locales in more lines? My brain hurts reading that long line...

@PikachuEXE i guess thats a no. So could u pretty please use your ⚡️ to approve

Copy link
Collaborator

@PikachuEXE PikachuEXE left a comment

Choose a reason for hiding this comment

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

Long horizontal lines sucks...

@PrestonN PrestonN merged commit 997b6c2 into FreeTubeApp:development Jul 8, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jul 8, 2022
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.

None yet

5 participants