Skip to content

Decouple responsibilities #7

Merged
merged 2 commits into from Oct 17, 2011

4 participants

@bob-p
Mint Digital member
bob-p commented Oct 13, 2011

This splits out the various parts of pathways meaning that for example mongo is not always required. Should fix issue 1.

@dstrelau

Don't you need the Mongo connection in Parser too?

@simonreed

Yes you will need the connection Parser. I think it would be cleaner if Server requires Parser and the dependencies move to parser.rb

@bob-p
Mint Digital member
bob-p commented Oct 14, 2011

Yes I think you do, Parser isn't actually loading at all atm because RAILS_ROOT is undefined....that was my next job

@rodreegez rodreegez merged commit ca11dfc into mintdigital:master Oct 17, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.