ejplatform / ej-server

EJ uses gamification and machine learning to promote discussions and participation in a virtual democracy
https://ejplatform.github.io/ej-server/
Other
38 stars 17 forks source link

Botões APROVE e DESAPROVE não tem string de tradução nos POs #552

Closed ricardopoppi closed 6 years ago

ricardopoppi commented 6 years ago

Me refiro aos botões da fila de moderação de comentários na moderação da conversa. Ver print:

captura de tela de 2018-10-11 12-25-12

ricardopoppi commented 6 years ago

@caiosba estou passando essa issue pra vc porque é parte do código que vc fez. Precisamos ajustar as strings de tradução no arquivo src/ej_conversations/jinja2/ej/role/comment-moderate.jinja2 pois não estão sendo geradas no PO. O ajuste teria que ser feito nessa parte:

        <ul class="ConversationComment-actions">
            <li up-expand>
                <input type="submit" class="Button" name="vote" value="approve"/>
                <span>{{ _('Approve') }}</span>
            </li>
            <li up-expand>
                <input type="submit" class="Button" name="vote" value="disapprove"
                        onClick="var $r = $('#Comment-{{ comment.id }}-rejection_reason'); return $r.is(':visible') ? true : $r.show() && false"/>
                <span>{{ _('Disapprove') }}</span>
            </li>
        </ul>
        <div id="Comment-{{ comment.id }}-rejection_reason" class="Comment-rejection" style="display: none;">
            <h3>{{ _('Reason') }}</h3>
            <p>{{ _('The comment will receive your rejection reason') }}</p>
            <select name="rejection_reason" id="Comment-{{ comment.id }}-rejection_reason">
              {% for k, v in rejection_reasons.items(): %}
                  <option value={{ k }}>{{ v }}</option>
              {% endfor %}
            </select>
            <input type="submit" class="Button" name="vote" value="disapprove"/>
        </div>
caiosba commented 6 years ago

Pronto para revisão: https://github.com/ejplatform/ej-server/pull/577.

caiosba commented 6 years ago

552