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

Add gift certs and banners as resources #103

Merged
merged 2 commits into from
Sep 11, 2015

Conversation

PreciselyAlyss
Copy link
Contributor

Update the ruby gem to add gift certificates and banners as resources.

@bookernath
Copy link
Contributor

These resources are not currently countable, the api/v2/banners/count returns a 404.

@aleachjr
Copy link

aleachjr commented Sep 9, 2015

Travis doesn't like you 😄

@PreciselyAlyss
Copy link
Contributor Author

Travis doesn't like me and I have some incorrect things in the resource files and the banner example that I saw after I submitted the PR

Bigcommerce.configure do |config|
config.auth = 'legacy'
# You will get this url when registering for an API key
config.url = ENV.fetch['BC_STORE_HASH']
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should probably be (), as fetch is a method call.

So you can do it either as
ENV.fetch 'BC_STORE_HASH'
or
ENV.fetch('BC_STORE_HASH')

@PreciselyAlyss PreciselyAlyss changed the title Add gift certs and banners as resources RFC: Add gift certs and banners as resources Sep 9, 2015
@PreciselyAlyss
Copy link
Contributor Author

Still need to add documentation/comments with options for enum fields to examples.

@PreciselyAlyss PreciselyAlyss force-pushed the 0.1.0 branch 2 times, most recently from 699586d to cc1e127 Compare September 11, 2015 00:53
@xyzjace
Copy link
Contributor

xyzjace commented Sep 11, 2015

👍 💚

@xyzjace xyzjace changed the title RFC: Add gift certs and banners as resources Add gift certs and banners as resources Sep 11, 2015
xyzjace added a commit that referenced this pull request Sep 11, 2015
Add gift certs and banners as resources
@xyzjace xyzjace merged commit 837e14e into bigcommerce:master Sep 11, 2015
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.

5 participants