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

Upgrade the app to Rails 5 #170

Merged
merged 37 commits into from
Apr 20, 2019
Merged

Upgrade the app to Rails 5 #170

merged 37 commits into from
Apr 20, 2019

Conversation

rsmithlal
Copy link
Contributor

In order to take advantage of the numerous performance improvements and security enhancements of Rails 5, an upgrade is required. Once we are on Rails 5, we can also then start integrating the social features that are provided as part of my Better Together project.

Had to remove a number of gems to get this to complete successfully. Some of these are requirements and must be added back in some way, but some may be able to be removed entirely. 

Removed gems: pg_search, activerecord-postgis-adapter, pg_search, jquery-rails, jquery-cookie-rails, jquery-turbolinks, paranoia, mailboxer, simple_form, datetimepicker-rails, rails-i18n, activeadmin, actionpack-action_caching, render_anywhere, web_console, rspec_rails, capybara, devise_invitable, haml-rails, and bootstrap-sass-extras
@rsmithlal
Copy link
Contributor Author

First cut: Had to remove a number of gems to get this to complete successfully. Some of these are requirements and must be added back in some way, but some may be able to be removed entirely.

Removed gems: pg_search, activerecord-postgis-adapter, pg_search, jquery-rails, jquery-cookie-rails, jquery-turbolinks, paranoia, mailboxer, simple_form, datetimepicker-rails, rails-i18n, activeadmin, actionpack-action_caching, render_anywhere, web_console, rspec_rails, capybara, devise_invitable, haml-rails, and bootstrap-sass-extras

@rsmithlal
Copy link
Contributor Author

I'm happy to have the travis build finally pass for this branch! I will be going through the app and testing each page and action to ensure that I catch as many hard failures as possible.

@joatu joatu temporarily deployed to joatuv2-pr-170 April 2, 2019 01:35 Inactive
@joatu joatu temporarily deployed to joatuv2-pr-170 April 2, 2019 02:08 Inactive
@rsmithlal rsmithlal temporarily deployed to joatuv2-pr-170 April 2, 2019 02:19 Inactive
@rsmithlal rsmithlal requested a deployment to joatuv2-pr-170 April 5, 2019 20:21 Abandoned
@rsmithlal rsmithlal closed this Apr 5, 2019
@rsmithlal rsmithlal reopened this Apr 5, 2019
@joatu joatu temporarily deployed to joatuv2-pr-170 April 5, 2019 20:22 Inactive
@joatu joatu temporarily deployed to joatuv2-pr-170 April 5, 2019 21:03 Inactive
@rsmithlal rsmithlal temporarily deployed to joatuv2-pr-170 April 5, 2019 21:31 Inactive
@rsmithlal rsmithlal temporarily deployed to joatuv2-pr-170 April 8, 2019 17:46 Inactive
@joatu joatu temporarily deployed to joatuv2-pr-170 April 20, 2019 14:53 Inactive
@rsmithlal rsmithlal temporarily deployed to joatuv2-pr-170 April 20, 2019 18:39 Inactive
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