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

Feature/data provider api #20

Merged
merged 15 commits into from
Jan 22, 2014
Merged

Feature/data provider api #20

merged 15 commits into from
Jan 22, 2014

Conversation

jdjkelly
Copy link
Owner

No description provided.

matcher - this was producing deprecation warnings
as a Concern-style module, and introducing the 
data_provider_for model DSL. We can now include the Concern
and use data_provider_for :x, :y, :z in our API provider 
classes. This will automatically execute matching methods 
to those symbols on create, otherwise raising a NotImplemented 
exception. This is the underpinning for making API
additions fast, and consistent. In the future, we should only 
have to define the internals of each symboled type of resource
provided to data_provider_for
config to get rid of deprecation error
unit - in case we don't know the initial unit
the specs can run for each individual API provider
jdjkelly added a commit that referenced this pull request Jan 22, 2014
@jdjkelly jdjkelly merged commit 196a359 into staging Jan 22, 2014
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.

1 participant