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

[Register add-on in translation system]: instantTranslate #116

Closed
1 task done
beqabeqa473 opened this issue Mar 24, 2024 · 16 comments
Closed
1 task done

[Register add-on in translation system]: instantTranslate #116

beqabeqa473 opened this issue Mar 24, 2024 · 16 comments
Labels
enhancement New feature or request

Comments

@beqabeqa473
Copy link

Add-on name

instantTranslate

Add-on maintainer

beqabeqa473

Stable branch

  • Stable branch exists in add-on repo
@beqabeqa473 beqabeqa473 added the enhancement New feature or request label Mar 24, 2024
Copy link

The pot files were built from the stable branch.

@CyrilleB79
Copy link
Contributor

Hi @beqabeqa473

But the add-on is actually already in the translation system.
And there is a fork in nvdaaddons; or actually yours is the fork.

Could you clarify why you have opened this ticket?

@beqabeqa473
Copy link
Author

it was disabled because of error in translations and @seanbudd asked me to create an issue.

@nvdaes
Copy link
Collaborator

nvdaes commented Apr 24, 2024

Now I have access to the l10n server to register add-ons.
@seanbudd ,if I should do anything for this add-on, let me know.

@seanbudd
Copy link
Member

@beqabeqa473 - can you please link information to this communication. I'm not sure what you are referring to

@beqabeqa473
Copy link
Author

@seanbudd
Copy link
Member

Thanks - @nvdaes - you can reenable this by uncommenting the relevant line in automatic.crontab. Ensure to push any changes to the crontab to this repo. I would encourage testing that the translation command works for the add-on by running it in the console before committing any changes.

@seanbudd
Copy link
Member

@nvdaes
Copy link
Collaborator

nvdaes commented Apr 30, 2024

@nvdaes - you can reenable this by uncommenting the relevant line in automatic.crontab. Ensure to push any changes to the crontab to this repo. I would encourage testing that the translation command works for the add-on by running it in the console before committing any changes.

Thanks @seanbudd. I'll do it after job, in about 9 or 10 hours.
Before registering add-ons, I usually clone them and do: scons pot mergePot
After registration I run the following command logged in the server:

/home/nvdal10n/mr/scripts/addonTranslationUpdates.sh instantTranslate, but I don't get any relevant feedback to see if automatic.crontab is really working.

@seanbudd
Copy link
Member

That process seems fine to me - any issues with crontab will get emailed out when it happens

@nvdaes
Copy link
Collaborator

nvdaes commented Apr 30, 2024

Thanks, years ago I received a lot of emails from Cron Daemon. I don't know if I'm subscribed there, but if you think that this is working, it's OK for me.
I'm noticing that translators seem not to add new registered add-ons to translate. Seems that Assembla is not sending notifications when add-ons are registered or when the documentation is submitted to SVN.

@nvdaes
Copy link
Collaborator

nvdaes commented Apr 30, 2024

Hello, seems that messages cannot be merged properly.

This is the error when I try to perform the corresponding automatic.crontab command:

 committing changes                                                              
 Sending        ko/add-ons/instantTranslate/nvda.po                              
 Transmitting file data .done                                                    
 Committing transaction...                                                       
 Committed revision 79705.                                                       
 ky wants instantTranslate: 0                                                    
 lt wants instantTranslate: 0                                                    
 mk wants instantTranslate: 0                                                    
 mn wants instantTranslate: 1                                                    
 Already available for translation, merging in new messages.                     
 svn status                                                                      
 mn/add-ons/instantTranslate/nvda.po:207: duplicate message definition...        
 mn/add-ons/instantTranslate/nvda.po:205: ...this is the location of the first de
 finition                                                                        
 msgmerge: found 1 fatal error                                                   
 mr addon2svn: command failed                                                    

@beqabeqa473, can you fix this in SVN repo?
Thanks.

@nvdaes
Copy link
Collaborator

nvdaes commented May 6, 2024

I'm trying to fix failures in this add-on. I'll remove the mn po file from the stable branch of GitHub and try to provide messages again for this language. If this doesn't work I'll revert changes.

@nvdaes
Copy link
Collaborator

nvdaes commented May 6, 2024

Seems there's nothing I can do to fix this. The mentioned file is not present. Sorry.

@nvdaes
Copy link
Collaborator

nvdaes commented May 6, 2024

Finally, seems that I have been able to fix the issue with this add-on, and now it contain l10n updates in the stable branch of the fork in nvdaaddons.
Let's wai until Friday to confirm that automatic.crontab works as expected.

@nvdaes
Copy link
Collaborator

nvdaes commented May 11, 2024

I'll close this now as completed: new translations have been received, available on the stable branch of the nvdaaddons fork, and the server hasn't notified about errors.

@nvdaes nvdaes closed this as completed May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants