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

Hateoas #7

Merged
merged 23 commits into from Jul 8, 2016

Conversation

Projects
None yet
1 participant
@vcolavin
Owner

vcolavin commented Jul 8, 2016

Nice.

vcolavin added some commits Jun 30, 2016

Somehow gets url_for(@location) and location_path working
Small commit, but it's a doozy with interesting and important implications. And metaprogramming.

Essentially, my main goal was to get url_for(@location) working, as well as location_path(@location). Rails doesn't easily let you mess with path helpers, but you can metaprogram your way into them.

This was way less painful than I expected, but now there is some untested and very brittle code in routes.rb. I'm gonna try and test it now.
Move action logic from JBuilder to presenter
Also adds logic in presenter to ensure that impossible actions (e.g. moving off the map) is not presented as an option.
All tests pass, especially the hateoas ones
- Aggressively clean database after tests
-

@vcolavin vcolavin merged commit 02550da into master Jul 8, 2016

@vcolavin vcolavin deleted the hateoas branch Jul 8, 2016

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