去年の年末に大掃除も年賀状もやらずに Rails に Pull Request を出してた。年末だけど大掃除もせず年賀状も書かず実家にも帰省せず rails/rails に Pull Reque...

去年の年末に大掃除も年賀状もやらずに Rails に Pull Request を出してた。

ActionMailer のプレビューで locale が複数ある場合に指定できるようにするというもの。 Kaizen Platform の Rails アプリにはこの機能付いてて多言語対応のメールをプレビューするときにめっちゃ便利だった。調べたところ Rails 4 時代にそういう Pull Request 出してた人がいて Merge 寸前まで行ってたんだけど commit が複数に分かれてたのを「 squash してくれない?」とレビューされたところでプルリク主の意欲が燃え尽きたっぽくて Merge されずにコンフリクトして死んでた。

Rails 5 でも動くようにコンフリクトを解消してテストケースも追加したのが以下。

動作イメージはこんな感じ。

34454066\-f8bf06ec\-eda5\-11e7\-82ba\-1c2a0961b6b8\.gif \(833×768\)

ただ Merge 後にバグってるのを指摘されていま直してるところです。

頭良くないのでこういうしょぼい Pull Request でしか contribute できないけど自分にできる範囲で貢献していきたい。

追記 2018-01-24

問題を修正する Pull Request も Merge してもらったんで多分 Rails 5.2 にこの機能入ります。

この記事に似ている記事

  • Screenshot
    Lokka の wysiwyg エディターは jQuery ベースの jwysiwyg といつやつが採用されている。しかしここ最近はメンテナンスされてなくて、最近の Chrome では利用できない状況だったりする。以前、ファイルアップロード機...
  • Bd5abb35faa37797ba4f11bf41e051a1
    前書いてた記事の続き。Kaizen Platform 時代は Naoya Ito さんの以下の記事にあるような感じで deploy してた。 Slack 上で hubot に話しかけると deploy 用の Pull Request が作られ...
  • 39116344152 1a76fb9898 b
    以下の文章は正月に「2017 年の Lokka へのコントリビュート目標」というタイトルで書いたまま下書きになってたものです。もう 2017 年も終わりそうだけど公開しておきます。RubyKaigi 2016 で komagata さんと L...
  • 1059 kaizen chat
    Kaizen Chat とはKaizen Platform 内でユーザー同士がコミュニケーションを取ることができるサービス。Kaizen Platform のユーザーカスタマーKaizen Platform と契約し、 A/B テストツールや...
  • 36176e0c deff 11e4 9bac 72b4e5ba15c7
    Rails とかで例外が発生したときに Slack に通知するやつ作った。 exception_notification という便利 gem のプラグインとして動く。実は ExceptionNotification 本体に SlackNoti...
  • Screenshot
    このブログ( Lokka )の DB は MySQL を使っている。 MySQL のバージョンは 5.7 なので、 column encoding を utf8mb4 、 collation を utf8mb4_general_ci とかにし...

Comments


(Option)

(Option)