Closed johangiraud closed 6 years ago
I'd like to make the button visible for editors (already running 😉) + for the current translator. For instance, if I'm not an editor, I start translating an article and I submit it. If I visit the VO page, I want to see the button like that: How can I discriminate this user: the translator of the article? @vinyll @davidbgk
Maybe somethin like translator.is_me()
?
@vinyll Also need your help here (see comment above)
@vinyll is it linked to https://github.com/cafebabel/cafebabel.com/issues/458 ?
First of all we would need a title
on these buttons. I personnally don't know the color codes and I still need to read the URL it will go to to guess if it's a translation that exists, if it's the original article or if it's to be created.
To answer your original question: it's a new little feature to put in place. We would then need something like a helper that would detect if the user is the translator of a certain article in a certain language.
First of all we would need a title on these buttons. I personnally don't know the color codes and I still need to read the URL it will go to to guess if it's a translation that exists, if it's the original article or if it's to be created.
Ok ! https://github.com/cafebabel/cafebabel.com/pull/487/commits/277533f5cf1401c6a946f166bac4d2b5442eb604
To answer your original question: it's a new little feature to put in place. We would then need something like a helper that would detect if the user is the translator of a certain article in a certain language.
Yep, that's what we have to do. For now it is only visible for editors. But we need it to be visible for the only translator.
Can you make an issue with that and ping @ffameli for prioritizing?
For François and me, it was already planned like that.
Can you help me fixing the tests? I tried to add title="Translate this article in fr"
but it didn't work :-(
Can you push your test fix trial to see what is breaking?
Les traductions en cours ne sont visbiles que pour le traducteur qui l'a réservé + l'ensemble des éditeurs. Alors pour ces profils seulement une pastille rose doit les prévenir que la traduction est réservée dans la langue concernée. Il ne me semble pas que ce soit une nouvelle fonction, mais une mise en forme de la privatisation des traductions. @vinyll confirmes tu ?
Hey @vinyll voilà tu sais c'était cette PR, j'avais déjà dit d'afficher le lien vers le draft en cours uniquement pour les éditeurs et je te demandais comment rendre visible le lien aussi pour celui qui a créé la traduction. Comme tu avais répondu et tout je pensais que c'était clair pour toi. ;-) Que proposes-tu ?
et aussi je n'arrive pas à fixer le test, je peux te demander de l'aide @vinyll
@vinyll
Bon je supprime ces tests @vinyll
Plutot que de supprimer ces tests, il vaut mieux désactiver le controle des tests: https://github.com/cafebabel/cafebabel.com/settings/branches/master et autoriser quand un test ne passe pas et que l'application ne répond pas comme attendu. Ca laissera comprendre pourquoi ca casse le jour où on a des régressions plutôt que d'être en aveugle ;)
D'accord, merci !
fix #458