Skip to content

Instantly share code, notes, and snippets.

@floehopper

floehopper/2017-05-24-manuals-publisher-update.md

Last active May 24, 2017 15:31
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save floehopper/f2b78ce2039f244aa3f3163e32fbb379 to your computer and use it in GitHub Desktop.
Save floehopper/f2b78ce2039f244aa3f3163e32fbb379 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
2017-05-24-manuals-publisher-update.md

Done

  • Fixed bug caused by upgrade to Mongoid v4 in combination with legacy data
  • Refactored Section to significantly simplify it
  • Standardized interfaces to service classes
  • Simplified service classes by inlining lots of one-line methods
  • Use instances of Manual vs ManualRecord in a bunch of Rake tasks
  • Avoid saving model(s) if call to Publishing API fails
  • Simplified Manual#initialize & #update
  • Fix ActionController::UrlGenerationError exception occuring in production

Doing/planning

  • Upgrading to Rails v5.0.3 & Mongoid v6.1.0
  • Possibly upgrading to Rails v5.1.x
  • Investigate avoiding Publishing API exceptions
  • Spike on "fully-migrated" version of app without local database
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment