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

Moved AR include into engine.rb file #12

Merged
merged 5 commits into from
Nov 27, 2011
Merged

Moved AR include into engine.rb file #12

merged 5 commits into from
Nov 27, 2011

Conversation

coryschires
Copy link
Contributor

This should be more inline with the current setup. I've had to change extend to include in order to have the class methods module mixed in properly, so that's a small difference.

@coryschires
Copy link
Contributor Author

Just checking in. I'd like to stop using my fork. Anything I can/should do to improve this pull request?

…ssions. Also, Had to do some work to get the tests passing

Merge remote branch 'upstream/master'

Conflicts:
  test_app/Gemfile.lock
coryschires added a commit that referenced this pull request Nov 27, 2011
Merged in the counter caching code. Added docs for previous pull request concerning recording unique impressions
@coryschires coryschires merged commit 85a714f into charlotte-ruby:master Nov 27, 2011
@hahi
Copy link

hahi commented Mar 12, 2012

Welcome to Cheap New Era Caps Shop!We offer wholesale cheap new era caps,New Era 59fifty Caps,New Era Snapback Caps,Wholesale New Era Caps,Wholesale New Era Fitted Caps. Fast shipping, free tax,lowest price,authentic quality!

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