Petit bug d'édition de langage

sam · Mars 12, 2019, 1:42

Oh my:

app/services/post_alerter.rb

be5e889b1


      
          post = first_unread_post(user, post.topic) || post
          count = unread_count(user, post.topic)
          if count > 1
            I18n.with_locale(user.effective_locale) do
              opts[:display_username] = I18n.t('embed.replies', count: count)
            end
          end

This is so hacky, we got to clean this up and do the localization using these guys:

github.com/discourse/discourse

config/locales/client.en.yml

be5e889b1


      
          notifications:
            tooltip:
              regular:
                one: "1 unseen notification"
                other: "{{count}} unseen notifications"
              message:
                one: "1 unread message"
                other: "{{count}} unread messages"
            title: "notifications of @name mentions, replies to your posts and topics, messages, etc"
            none: "Unable to load notifications at this time."
            empty: "No notifications found."
            more: "view older notifications"
            total_flagged: "total flagged posts"
            mentioned: "<span>{{username}}</span> {{description}}"
            group_mentioned: "<span>{{username}}</span> {{description}}"
            quoted: "<span>{{username}}</span> {{description}}"
            replied: "<span>{{username}}</span> {{description}}"
            posted: "<span>{{username}}</span> {{description}}"
            edited: "<span>{{username}}</span> {{description}}"
            liked: "<span>{{username}}</span> {{description}}"

This file has been truncated. show original

And stop using a “pretend” username which is broken when you switch locales (cause its baked into the notification)

@gerhard can you clean this up and do the localization here instead:

discourse/app/assets/javascripts/discourse/widgets/notification-item.js.es6 at be5e889b1b55de148b32ae864390a585e027b1ec · discourse/discourse · GitHub

Sujet		Réponses	Vues
Can find "replies" string in the texts to translate Support	4	796	Mars 12, 2019
Mislocalization of "x replies" in notifications Bug	2	1308	Janvier 29, 2015
Translation missing for Reply Support	6	1655	Février 28, 2017
Notification error Bug	14	1685	Septembre 14, 2017
Translate some Entries Support	4	2154	Janvier 25, 2016

Petit bug d'édition de langage

Sujets connexes