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 Guide ideas and suggestions #1264

Closed
subvertallchris opened this issue Aug 11, 2016 · 4 comments
Closed

Upgrade Guide ideas and suggestions #1264

subvertallchris opened this issue Aug 11, 2016 · 4 comments

Comments

@subvertallchris
Copy link
Contributor

A running list of thoughts about http://neo4jrb.readthedocs.io/en/8.0.x/UpgradeGuide.html. Please comment!

@subvertallchris
Copy link
Contributor Author

subvertallchris commented Aug 11, 2016

The :server_db section is a little unclear. I think it would be helpful for it to provide before and after examples.

@ProGM ProGM added this to the 8.0.0 milestone Aug 11, 2016
@subvertallchris
Copy link
Contributor Author

subvertallchris commented Aug 11, 2016

We should include a note about including the migration rake task if not using the Railtie:

load 'neo4j/tasks/migration.rake'

https://github.com/neo4jrb/neo4j/blob/f2e93f0722868319675d1f983471d22951807b93/lib/neo4j/railtie.rb#L42

For that matter, we should probably have an entire section on upgrade steps required of people who skip the Railtie.

cheerfulstoic added a commit that referenced this issue Aug 19, 2016
@cheerfulstoic
Copy link
Contributor

Just pushed some changes (they'll need to build on RTD). Have a look!

ProGM added a commit that referenced this issue Aug 21, 2016
* 8.0.x:
  CHANGELOG entry for #1254
  Improvements based on #1264
  Destroy should return object like in ActiveRecord
  Use ActiveSupport::Logger instead of base Logger
  Release .15 patch with #1270 change
  Trying to fix ruby 2.0 build.
  Fix gemspec.
  Fix Gemfile for jruby.
  Accepting `nil` as enum value
  Add information about which exceptions are replacing old exceptions
  Update ActiveRel.rst
  Up to 7.1.2 to include fix from model_label_map_fix
  Put this back, because of course it broke things
  Fix issue with label / model mappings getting stuck before all models have a chance to load.  Also don't clear WRAPPED_CLASSES (I think this will fix the CypherNode issue we've been seeing in development
@cheerfulstoic
Copy link
Contributor

Closing for now, but comment / re-open if I missed anything

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

No branches or pull requests

3 participants