- Issue created by @miro_dietiker
- First commit to issue fork.
- @ranjit1032002 opened merge request.
- Status changed to Needs review
over 1 year ago 7:13am 3 May 2023 - 🇮🇳India Ranjit1032002
Created MR!2 for the issue mentioned, please review.
Thank you. - Status changed to Needs work
over 1 year ago 9:07am 3 May 2023 - 🇨🇭Switzerland miro_dietiker Switzerland
Thank you for giving this a try, i checked the original code:
The addRemoteMapping can not change, since we then would break identifying the item when translations return.
The addMessage should move to the job outside of the loop.
Supertext identifies item ids by just receiving GroupId (translation job item ids) and we don't need (and we don't get) a per-item remote id.
- Status changed to Needs review
over 1 year ago 11:52am 3 May 2023 - 🇮🇳India Ranjit1032002
@miro_dietiker I had made changes to the MR!2, please review.
Thank You. - Status changed to Needs work
over 1 year ago 3:02pm 14 September 2023 - 🇨🇭Switzerland berdir Switzerland
no need to add to the job item. no need to collect and group the ids, because it's always the same.