Skip to content

Conversation

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 25, 2025

Pull Request Type

  • Chore

Related issue

Closes #5000

Description

Updates all keys and values to use sentence casing to have consistent wording thought our application similar to YT. I still need to update the keys in all the other locale files and in the files where the keys are actually being used. Before i continue todo that i require some help, see Additional context section.

Testing

  • Check the file
  • Look around in the UI to check that sentence casing has been applied

Additional context

Help needed on this matter: @absidue @PikachuEXE @kommunarr @ChunkyProgrammer @MarmadileManteater

This PR is drafted because of what has been described in the Description section and the following.
IMO we have a problem how keys are being used in our files and i would like to address that in this PR (allot of keys are being updated anyway so why not kill two birds with one stone). We have no consistent naming convention on how a key should be written.

Examples taken from the file:

  • Normal writing? Next chapter:
  • Pascal Case, KeyboardShortcutPrompt:
  • Camel Case, shortcutLabelSeparator:
  • Full sentences (i guess same as the first one), This video is unavailable because of missing formats. This can happen due to country unavailability.:

Im in favor of using camel case so that its consistent with our codebase and a nice addition is that the keys will become smaller but ofcourse still be descriptive so everyone knows what that key is used for

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc changed the title Update Weblate keys and values to use sentence casing Oct 26, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc marked this pull request as ready for review November 2, 2025 19:26
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) November 2, 2025 19:26
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 2, 2025
auto-merge was automatically disabled November 2, 2025 19:26

Pull request was converted to draft

@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 2, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2025

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

Conflicts have been resolved. A maintainer will review the pull request shortly.

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

Labels

None yet

1 participant