Skip to content

Conversation

@djsegal
Copy link
Contributor

@djsegal djsegal commented Feb 26, 2016

Resolves: #546


I tried to match the create action to be consistent with the rest of the controllers.

I did however leave the show action in case anyone was still using it for legacy purposes

@djsegal djsegal force-pushed the djsegal/confirmation branch from f3c7ea8 to 73a4f85 Compare February 26, 2016 00:03
@djsegal djsegal force-pushed the djsegal/confirmation branch from 73a4f85 to e14bde9 Compare February 26, 2016 22:07
@djsegal
Copy link
Contributor Author

djsegal commented Jun 12, 2016

@booleanbetrayal:

@zachfeldman
Copy link
Contributor

Hi there @djsegal ,

In an effort to cleanup this project and prioritize a bit, we're marking pull requests that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this pull request. Obviously feel free to re-open it at any time if it's the right time or this was done in error! If you do, please rebase it with the latest master and explain why it's still needed.

We really appreciate your contribution, we're just trying to make this project manageable again to move it forward.

Hope all is well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants