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

Sync of docs to Transifex is not working #7658

Open
rakekniven opened this issue Dec 1, 2021 · 14 comments · Fixed by #7892
Open

Sync of docs to Transifex is not working #7658

rakekniven opened this issue Dec 1, 2021 · 14 comments · Fixed by #7892

Comments

@rakekniven
Copy link
Member

rakekniven commented Dec 1, 2021

Strings of new docs about groupware, specially calendar, are not available at Transifex.

Checked german the last three days. No changes are synced but people commited changes to GitHub.

Also the renaming from "pim" to "groupware" does not show up and "index" is old as well.

Bildschirmfoto 2021-12-01 um 16 18 41

@ChristophWurst
Copy link
Member

did that break because of #7633 perhaps?

@rakekniven
Copy link
Member Author

Not sure as I do not know details about the sync.

But even after rename the new resources should come up AFAIU.

@ChristophWurst
Copy link
Member

I've asked our sysadmins for help.

@rakekniven
Copy link
Member Author

Somebody started some action already.
Looks promising. Will report tomorrow in detail.

@rakekniven
Copy link
Member Author

Ok, that finally works. Last evening and night there has been a lot of changes.
The renaming has caused about 250 sentences to be untranslated for each language.
So for the future this has to be kept in mind.
For most of them it is just picking the previous translations.

Will do the new calendar strings the next days.

Closing here. Bye.

@rakekniven
Copy link
Member Author

After reinserting all previous translations for the strings I found out, that for the resource "calendar" the strings for new features like "appointment" are still not at Transifex.

@rakekniven rakekniven reopened this Dec 2, 2021
@rakekniven
Copy link
Member Author

rakekniven commented Dec 2, 2021

At docs.nextcloud.com they are present in english. See https://docs.nextcloud.com/server/latest/user_manual/de/groupware/calendar.html#appointments

That content is generated by content from GitHub, or?

Which branches are currently synced to Transifex?
EDIT: content for appointment feature is in master, stable23 and release23. Not in any 22 branches which makes sense.

@ChristophWurst
Copy link
Member

So for the future this has to be kept in mind.

My bad. Will do.

At docs.nextcloud.com they are present in english. See https://docs.nextcloud.com/server/latest/user_manual/de/groupware/calendar.html#appointments

That page is only deployed once a day. You don't see it update live. Was that your question?

@rakekniven
Copy link
Member Author

rakekniven commented Dec 2, 2021

You don't see it update live. Was that your question?

That wasn't a question. Just an information to support finding the cause.

To make it short:
Description of appointment feature is at GitHub in various branches and at docs.nextcloud.com but not at Transifex.
Even after the renamed resources have been synced last night.

What is the reason?

@ChristophWurst
Copy link
Member

@matthieugallien do you know?

@rakekniven
Copy link
Member Author

See https://docs.nextcloud.com/server/latest/user_manual/de/groupware/calendar.html#appointments after job ran last night.

At Transifex everything is 100% complete translated, but the appointment section is still english.

@rakekniven
Copy link
Member Author

Hey @CarlSchwan ,

Transifex notification just arrived. Thanks for this.
You closed this issue by mentioning #7892 . I would vote to reopen this issue to have issue for repairing/integrating GitHub action for it.
What do you think?

@rakekniven rakekniven reopened this Jan 10, 2022
@CarlSchwan
Copy link
Member

Ok, it's good that doing it manually worked. I will do a bit of testing to figure out why the action doesn't work anymore this week ;)

@christianlupus
Copy link
Contributor

Just for reference: I am still missing some changes. An example would be here, see also my comment in #7848. Note the missing double * in the translated version at the end.
The change was made by me before the manual intervention by @CarlSchwan so could you please have a look if there is still something missing/failing?

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

Successfully merging a pull request may close this issue.

5 participants