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

Jump to bottom

v2.0.0 #16

Merged
merged 9 commits into from Feb 4, 2018
Merged

v2.0.0 #16

merged 9 commits into from Feb 4, 2018

Conversation

tarnas14
Copy link
Contributor

  • Moved vouchers related methods to client.vouchers.* namespace
  • Moved redemptions related methods to client.redemptions.* namespace
  • Moved distributions related methods to client.distributions.* namespace
  • Renamed client.customer.* to client.customers.*
  • Removed outdated client.distributions.publish(campaignName) method interface
  • Fixed utils methods to accept vouchers with None gift
  • added some tests to check if the things are broken in the future
  • updated examples

@tarnas14
None gift no longer breaks price calculation utils
ef623fa
@tarnas14
added test harness for methods that will be moved to vouchers.* names…
b462595 
…pace
@tarnas14
added client.vouchers namespace
4946c22 
methods: list, get, create, update, enable, disable

these methods are still available under client.*, but are marked as
deprecated
@tarnas14
added client.redemptions namespace
3e8aab0 
methods: redeem, getForVoucher, list, rollback
method voucherify.redemptions renamed to voucherify.listRedemptions
methods redeem, redemption, rollback are still available under
voucherify.*, but are marked as deprecated
@tarnas14
moved publish method to client.distributions.* namespace
c5caa7a 
client.publish method no longer exists
@tarnas14
removed deprecated methods that were moved to redemptions and voucher…
a805dfe 
…s namespaces
@tarnas14
updated voucherify_test examples to new nemespaces
e9afa84
@tarnas14
moved client.customer.* to client.customers.* namespace
c37a1c9
@tarnas14
Changed readme structure to be consistent with nodejs sdk readme
6a29dda 
also added voucherify logo with python (TM) logo
@tarnas14 tarnas14 merged commit 9caeaf4 into master Feb 4, 2018
@tarnas14 tarnas14 deleted the namespace_revamp branch February 4, 2018 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tpindel tpindel Awaiting requested review from tpindel

@LukaszWiktor LukaszWiktor Awaiting requested review from LukaszWiktor

@imvanzen imvanzen Awaiting requested review from imvanzen

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@tarnas14