search

mirego / accent

The first developer-oriented translation tool. True asynchronous flow between translators and your team.
https://www.accent.reviews
BSD 3-Clause "New" or "Revised" License
1.29k stars 97 forks source link

Translator and Proofreader #393

Closed RappyTV closed 11 months ago

RappyTV commented 11 months ago

I would appreciate a feature where "Translator" and "Proofreader" are two seperate roles so when a normal translator changes a string, it's not deployed directly and has to first be approved by a Proofreader.

simonprev commented 11 months ago

Just pushed a new role "translator" and a new state "translated" on a strings that is made true when the translation is saved in the UI

RappyTV commented 11 months ago

@simonprev I really appreciate the effort but the "Reviews tab" is just gone after the update and I don't think it's supposed to be this way. As an admin: https://media.rappytv.com/u/opera_1Y3y5Q9N0T.png As a translator & as a reviewer: https://media.rappytv.com/u/firefox_DfuMEUmsyb.png

simonprev commented 11 months ago

Oh yeah sorry, it was a bug in v1.19.8 but it’s fixed in v1.19.9 😄

RappyTV commented 11 months ago

tyvm ❤️

RappyTV commented 11 months ago

Translators and reviewers still don't have access to the "Review tab". Admins do

simonprev commented 11 months ago

Oh you’re right, fixed in https://github.com/mirego/accent/commit/6b9f7c30f67067595d4323bbc67517416305c595 and building right now in v1.19.10

RappyTV commented 11 months ago

W