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

Fixes #153

Merged
merged 3 commits into from
Jul 8, 2022
Merged

Fixes #153

merged 3 commits into from
Jul 8, 2022

Conversation

boblail
Copy link
Member

@boblail boblail commented Jul 7, 2022

Hi friends! 👋 ❤️

I have a few small fixes for getting started with houston new:

  • fix: Replace redirect_to :back with redirect_back now that Rails has deprecated the former
  • fix: Handle ActiveRecord::NoDatabaseError in load_persistent_triggers.rb so that bin/setup doesn't fail before it can create the databases
  • fix: Replace MultiJson with JSON since the multi_json gem is no longer loaded by default

@boblail boblail requested review from kobsy and lukebooth July 7, 2022 21:44
Copy link
Contributor

@kobsy kobsy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes all make good sense. Thanks! 😁 👍

@boblail boblail merged commit b648e43 into master Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants