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

Automatic translation support adding translation as "approved" #9048

Closed
KadAnna opened this issue Apr 5, 2023 · 9 comments · Fixed by #9174
Closed

Automatic translation support adding translation as "approved" #9048

KadAnna opened this issue Apr 5, 2023 · 9 comments · Fixed by #9174
Assignees
Labels
backlog This is not on the Weblate roadmap for now. Can be prioritized by sponsorship. enhancement Adding or requesting a new feature. good first issue Opportunity for newcoming contributors. hacktoberfest This is suitable for Hacktoberfest. Don’t try to spam. help wanted Extra attention is needed.
Milestone

Comments

@KadAnna
Copy link

KadAnna commented Apr 5, 2023

Describe the problem

Following ticket #8979, I would like to have an option to set the automatically translated strings as "for review". This will correspond with the "for review" workflow.

Describe the solution you'd like

Add an option "Add as for review" to Automatic translation mode of the add-on set-up.

Describe alternatives you've considered

No response

Screenshots

No response

Additional context

No response

@nijel
Copy link
Member

nijel commented Apr 5, 2023

The strings should be added as “for review” right now. I thought you wanted to have the possibility to add strings as “approved”.

@nijel nijel added enhancement Adding or requesting a new feature. backlog This is not on the Weblate roadmap for now. Can be prioritized by sponsorship. labels Apr 5, 2023
@nijel nijel changed the title Automatic translation add-on can add new translation as "for review" Automatic translation add-on can add new translation as "approved" Apr 5, 2023
@github-actions
Copy link

github-actions bot commented Apr 5, 2023

This issue has been added to the backlog. It is not scheduled on the Weblate roadmap, but it eventually might be implemented.

In case you need this feature soon, please consider helping or push it by funding the development.

@KadAnna KadAnna changed the title Automatic translation add-on can add new translation as "approved" Automatic translation add-on can add new translation as "for review" Apr 5, 2023
@KadAnna
Copy link
Author

KadAnna commented Apr 5, 2023

I just tried tool Automatic translation. There the Automatic translation mode Add as translation (together with Enabled reviews Setting) sets strings as for review. So it this is the same at the addon Automatic translation, then this ticket is unnecessary. Just that because of the bug #8979 I could not see it.

@nijel
Copy link
Member

nijel commented Apr 12, 2023

I still think it makes sense to add an option to add string as approved, so let's keep this issue open.

@nijel nijel added hacktoberfest This is suitable for Hacktoberfest. Don’t try to spam. help wanted Extra attention is needed. good first issue Opportunity for newcoming contributors. labels Apr 12, 2023
@nijel
Copy link
Member

nijel commented Apr 12, 2023

To implement this:

@nijel nijel changed the title Automatic translation add-on can add new translation as "for review" Automatic translation add-on can add new translation as "approved" Apr 12, 2023
@nijel nijel changed the title Automatic translation add-on can add new translation as "approved" Automatic translation support adding translation as "approved" Apr 12, 2023
@github-actions
Copy link

This issue seems to be a good fit for newbie contributors. You are welcome to contribute to Weblate! Don't hesitate to ask any questions you would have while implementing this.

You can learn about how to get started in our contributors documentation.

@Krishhhnaa
Copy link

is this issue closed ?

@nijel
Copy link
Member

nijel commented Apr 12, 2023

No, it is open.

@nijel nijel linked a pull request May 3, 2023 that will close this issue
5 tasks
@nijel nijel added this to the 4.18 milestone May 3, 2023
nijel pushed a commit that referenced this issue May 3, 2023
* Add auto translation as "approved" (Closes: 9048)

* Only show approved choice if user has the permission

* fixup! Only show approved choice if user has the permission
@github-actions
Copy link

github-actions bot commented May 3, 2023

Thank you for your report; the issue you have reported has just been fixed.

  • In case you see a problem with the fix, please comment on this issue.
  • In case you see a similar problem, please open a separate issue.
  • If you are happy with the outcome, don’t hesitate to support Weblate by making a donation.

@nijel nijel self-assigned this May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog This is not on the Weblate roadmap for now. Can be prioritized by sponsorship. enhancement Adding or requesting a new feature. good first issue Opportunity for newcoming contributors. hacktoberfest This is suitable for Hacktoberfest. Don’t try to spam. help wanted Extra attention is needed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants