issue投げよう

昨日の自分へ

action_argsとactive_adminが競合してる気がするのであとでちゃんと検証してフィードバックとかしたい

— ふかやつ (@fukayatsu) November 26, 2012

発端はこれ。
プライベートで作業してるときに、両方のgemを入れてるとactive_adminの管理画面にログインした後に
wrong number of arguments
とか表示されるのに気づいた。

エラーを追うと、

active_admin/lib/active_admin/page_controller.rb at master · gregbell/active_admin · GitHub

module ActiveAdmin

  # All Pages controllers inherit from this controller.
  class PageController < BaseController

    # Pages::Page subclasses Page::Base which implements the layout code
    layout false

    actions :index

    def index(options={}, &block)
      render "active_admin/page/index"
    end

    def clear_page_actions!
      active_admin_config.clear_page_actions!
    end
  end
end

この11行目がaction_argsと競合しているらしかった。

** というのをasakusarb/action_argsのissuesに書けよ俺！ **

次からそうしますm(_ _)m

今回はたまたま@a_matsudaさんが最初のtweetを捕捉して頂いたらしく、速攻で修正してくださいました。ありがとうございました!!

@fukayatsu 修正してみたので、よければ先っちょで試してみていただけますでしょうか？ github.com/asakusarb/acti… (明日ぐらいにはgemにもすると思いますが)

— Akira Matsudaさん (@a_matsuda) 11月 26, 2012

@a_matsuda おおお、ありがとうございます！！修正箇所と正常に動作することを確認しました。ちゃんとフィードバックしなくてすみませんm(_ _)m

— ふかやつさん (@fukayatsu) 11月 26, 2012

@fukayatsu よかったです :) こちらはこのバグにはまったく気づけてなかったところでたまたまさっきのツイートを拝見したので助かりました。ありがとうございました。

— Akira Matsudaさん (@a_matsuda) 11月 26, 2012

その後、他にも変更が入ってgemのバージョンがv0.0.3 => v1.0.0になったみたいです。

action_args | RubyGems.org | your community gem host

結論

みんなちゃんとissue投げるところからOSSへの貢献をしよう。 #cwt2012 twitpic.com/bazq7r

— Aki (@nekoruri) November 7, 2012