search

kalenjordan / magehero

Magento ninjas, rockstars, and samurais
http://magehero.com
MIT License
39 stars 22 forks source link

Clicking "Post" in logged-in view inserts record to DB #106

Open philwinkle opened 9 years ago

philwinkle commented 9 years ago

We need a blank form for Creating a new post that is not explicitly an edit route. This was a creative way to save time while bootstrapping but it's very easy to fill up the db with a bunch of posts that all contain "New Post" as the subject and nothing else.

kalenjordan commented 9 years ago

This man speaks truth.

Truth: this man speaks it. It: the truth which this man speaks. Speaks: this man does it with truth. Man: the person from whom truth is being spoken.

bobbyshaw commented 9 years ago

This sounded familiar. I actually already PR'd this: https://github.com/kalenjordan/magehero/pull/70, it got reverted because of an issue saving tags which I then fixed. How would you like to proceed @kalenjordan ?

kalenjordan commented 9 years ago

Could we get that just tested on staging real quick and then I'll push it?

philwinkle commented 9 years ago

If only we had unit tests..... On Feb 4, 2015 8:31 AM, "kalenjordan" notifications@github.com wrote:

Could we get that just tested on staging real quick and then I'll push it?

— Reply to this email directly or view it on GitHub https://github.com/kalenjordan/magehero/issues/106#issuecomment-72854170 .

bobbyshaw commented 9 years ago

kalenjordan commented 9 years ago

RT @bobbyshaw's last comment. @philwinkle