I'm positive to adding such generic option. I18n already have the `transalate!` method to raise an error if the translation is missing maybe we should use it in the Rails wrappers?
On Mon, May 23, 2016 at 7:59 PM Kevin Sjöberg <kev.sjob...@gmail.com> wrote: > I'm on a mission to unify the logic for raise_on_missing_translation between > controllers and views. Currently this is scoped to ActionView. I'm > proposing that we move the config option to the I18n scope, giving us > config.i18n.raise_on_missing_translations and thus deprecating > config.action_view.raise_on_missing_translations. > We then update AbstractController#translate and > TranslationHelper#translate to use the new config option and raise errors > accordingly. > > Since I'm not usually contributing to Rails I might overlook some > technical or architectural aspects of the problem. Would love some feedback > on the approach I'm taking. If all sounds good, I will begin working on a > pull request. > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to rubyonrails-core+unsubscr...@googlegroups.com. > To post to this group, send email to rubyonrails-core@googlegroups.com. > Visit this group at https://groups.google.com/group/rubyonrails-core. > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-core+unsubscr...@googlegroups.com. To post to this group, send email to rubyonrails-core@googlegroups.com. Visit this group at https://groups.google.com/group/rubyonrails-core. For more options, visit https://groups.google.com/d/optout.
