Add Flagging.with_flag(reason), as per README

Open dgilperez opened this issue 11 years ago • 3 comments

Hi!

I was trying to use the .with_flag(reason) scope for Flagging, as it is documented in README, but I realized there's no scope for that in the code. I added it

Also, I fixed the spec schema.rb to include flaggings_count column (all tests were failing before that).

Hope you like to include this!

Regards,

PS: I also noticed generator spec is red in Rails >= 4.0, but I don't know how to fix that. You can have a peep here: https://travis-ci.org/dgilperez/make_flaggable/builds/35460558

Sep 16 '14 18:09 dgilperez

I also added reason option to flagged? and unflag, as per README as well.

Sep 18 '14 09:09 dgilperez

Any chance of this being merged?

Dec 22 '14 22:12 dgilperez

Hey @medihack, do you think this have a chance of getting merged anytime soon? Thanks!

Mar 20 '15 23:03 dgilperez