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

Add Slovenian (sl) l10n #143

Merged
merged 2 commits into from
Sep 3, 2024
Merged

Add Slovenian (sl) l10n #143

merged 2 commits into from
Sep 3, 2024

Conversation

filmsi
Copy link
Contributor

@filmsi filmsi commented Sep 2, 2024

Courtesy of Slovenian GNOME l10n team.

Thank you.

BTW, adding files via Gnome Damned Lies is not working for Cassette, upon uploading the po file it responds with the following error (this means Cassette might not end localized into many languages):
»[Errno 128] Command: ‘['git', 'push', 'origin', 'main']’, Error: Warning: Permanently added 'github.com' (ED25519) to the list of known hosts. ERROR: Permission to Rirusha/Cassette.git denied to gnomeinfra-github. fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists. «.

Add Slovenian l10n to Cassette
Created sl.po, fully localized
@fiersik
Copy link
Contributor

fiersik commented Sep 2, 2024

@filmsi what is strange about banning direct writes to the repository? All changes are only made through pr

@Rirusha
Copy link
Owner

Rirusha commented Sep 2, 2024

The problem is known. There was some confusion in providing access to the repository.

https://gitlab.gnome.org/Teams/Translation/Coordination/-/issues/164#note_2213268

@Rirusha
Copy link
Owner

Rirusha commented Sep 2, 2024

Thanks for the translation, but I'm sorry. I have agreed that the translation will not be accepted through MR/PR in the issue mentioned above. Please add the translation as soon as GNOME Infrastructure accepts the invitation to write to this repository.

@Rirusha Rirusha closed this Sep 2, 2024
@filmsi
Copy link
Contributor Author

filmsi commented Sep 2, 2024

So this is how you support l10n?

@Rirusha
Copy link
Owner

Rirusha commented Sep 3, 2024

I'm not sure what the technical consequences might be; there must be a reason why they have this rule.

Though I realized I made the wrong decision yesterday, it’s not wise to rely on something that isn't working yet.

I'll reopen the PR. Thanks for the translation!

@Rirusha Rirusha reopened this Sep 3, 2024
@Rirusha Rirusha merged commit 968669d into Rirusha:main Sep 3, 2024
2 checks passed
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.

3 participants