search

okfnfr / dada-france-theme

Le theme alaveteli utilisé par Ma Dada
https://madada.fr
MIT License
4 stars 3 forks source link

Traduire notification delivered #60

Closed pzwsk closed 4 years ago

pzwsk commented 4 years ago

Le texte affiché "Delivered" est une réponse du serveur et ne peut donc être traité comme les autres contenus textes à traduire.

J'ai essayé de modifier le code ci-dessous dans le fichier https://github.com/okfnfr/dada-france-theme/blob/staging/lib/views/request/_outgoing_correspondence.html.erb

           <% if delivery_status %>
        <div class="correspondence__header__delivery-status">
          <%= link_to outgoing_message_delivery_status_path(outgoing_message), :class => "toggle-delivery-log toggle-delivery-log--#{outgoing_message.delivery_status.simple} js-toggle-delivery-log", :'data-delivery-status' => outgoing_message.delivery_status.simple do -%>
                <%= outgoing_message.delivery_status.simple.to_s.humanize -%>
          <% end -%>
        </div>
      <% end %>
    </div>

par 

      <% if delivery_status %>
        <div class="correspondence__header__delivery-status">
          <%= link_to outgoing_message_delivery_status_path(outgoing_message), :class => "toggle-delivery-log toggle-delivery-log--#{outgoing_message.delivery_status.simple} js-toggle-delivery-log", :'data-delivery-status' => outgoing_message.delivery_status.simple do -%>
              <% if outgoing_message.delivery_status.simple == 'delivered' %>
                  <%= {{delivered}} -%>
              <% else %>
                  <%= outgoing_message.delivery_status.simple.to_s.humanize -%>
              <% end %>
          <% end -%>
        </div>
      <% end %>
    </div>

mais il semble y avoir une erreur.

@laurentS si tu peux m'aider là dessus.

Merci

laurentS commented 4 years ago

Remplace <%= outgoing_message.delivery_status.simple.to_s.humanize -%> par <%= outgoing_message.delivery_status.to_s!.humanize -%> et ça semble marcher.

laurentS commented 4 years ago

J'ai fait les modifications décrites dans mon précédent message. Ca fonctionne sur staging. je ferme ce ticket.

pzwsk commented 4 years ago

Excellent, merci Laurent.

On Wed, Dec 4, 2019 at 10:22 AM Laurent Savaete notifications@github.com wrote:

J'ai fait les modifications décrites dans mon précédent message. Ca fonctionne sur staging. je ferme ce ticket.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/okfnfr/dada-france-theme/issues/60?email_source=notifications&email_token=AASEVR7ATVYDMXEPB3BTS5TQW5ZEHA5CNFSM4JPCBFFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF4JLQY#issuecomment-561550787, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASEVRYMIFA4CTCVDAX2JE3QW5ZEHANCNFSM4JPCBFFA .