Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent empty URL searches from erroring #255

Merged
merged 1 commit into from Jul 22, 2015
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jul 22, 2015

When a URL is not present, the controller would render the new template but without going through the new method it wouldn’t have all the template variables it needed.

  • Call the new method before rendering with a 422 status code

https://trello.com/c/OAJAO2RB/

When a URL is not present, the controller would render the `new`
template but without going through the new method it wouldn’t have all
the template variables it needed.

* Call the `new` method before rendering with a 422 status code
boffbowsh added a commit that referenced this pull request Jul 22, 2015
Prevent empty URL searches from erroring
@boffbowsh boffbowsh merged commit ecc1dff into master Jul 22, 2015
1 check passed
1 check passed
default "Build #267 succeeded on Jenkins"
Details
@boffbowsh boffbowsh deleted the allow-empty-searches branch Jul 22, 2015
@issyl0
Copy link
Contributor

@issyl0 issyl0 commented Jul 22, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.