voucherifyio / voucherify-ruby-sdk Goto Github PK

Add support for following API Endpoint:
https://docs.voucherify.io/reference#get-member-activities

At the moment we're wrapping around the outside of the Voucherify SDK to rescue + inspect the errors it raises to implement our own retry mechanism to deal with the odd timeout and very occasionally an odd network error or two.

Are there any plans to build a retry mechanism into the Ruby SDK à la Stripe's Ruby client?

See here for Stripe's implementation: https://github.com/stripe/stripe-ruby/blob/master/lib/stripe/stripe_client.rb#L451

If we were to put aside the time to move our retry mechanism into this gem would @voucherifyio be open to merging this in rather than us maintaining our own fork?

Make sure that the following methods are provided:

• Server side: Promotions.ListAllTiers(?is_avai=able=true/false); https://docs.voucherify.io/v2018-08-01/reference#list-promotion-tiers
• Client side: Promotions.ListAllTiers(?is_avai=able=true/false); https://docs.voucherify.io/v2018-08-01/reference#list-promotion-tiers-client-side

Make sure that the following methods can work with extended Order Item model:

• Order; https://docs.voucherify.io/v2018-08-01/reference#the-order-object

• Order Items; https://docs.voucherify.io/v2018-08-01/reference#the-order-item-object\

• Create Order; https://docs.voucherify.io/v2018-08-01/reference#create-order

• Update Order; https://docs.voucherify.io/v2018-08-01/reference#update-order

• Validate Voucher; https://docs.voucherify.io/v2018-08-01/reference#validate-voucher

• Validate Voucher (client side); https://docs.voucherify.io/v2018-08-01/reference#vouchers-validate

• Validate Promotions; https://docs.voucherify.io/v2018-08-01/reference#validate-promotions-1

• Redeem Voucher; https://docs.voucherify.io/v2018-08-01/reference#redeem-voucher

• Redeem Voucher (client side); https://docs.voucherify.io/v2018-08-01/reference#redeem-voucher-client-side

• Redeem Promotion; https://docs.voucherify.io/v2018-08-01/reference#redeem-promotion

Close the issue if no operation is needed or provide a solution with next release.

Maybe we shall consider adding utils method for handling signature verification.

require 'openssl'

  def verify_webhook
    calculated_hmac = OpenSSL::HMAC.hexdigest('sha256', SECRET, request.body.read)
    ActiveSupport::SecurityUtils.secure_compare(calculated_hmac, request.headers["X-Voucherify-Signature"])
  end

Percent-off vouchers with a cap amount set do not take that cap into consideration when calculating a discount.

From the util code here: https://github.com/voucherifyio/voucherify-ruby-sdk/blob/master/lib/voucherify/utils.rb#L37-L41 It's clear that the discount is a simple function of the percent off times the base price.

Happy to open a PR for this but I'm not sure of all the ways that a cap can come into play. Let me know if you need any more info or a reproduction case to move forward. Thanks!

Drop API version support for other versions than 2018-08-01 and latest

Consents namespace:

• List Consents: GET /consents - Docs
• List Consents (Client): GET /consents - Docs

Customers namespace:

• Update Customer's Consents: PUT /customers/{id}/consents - Docs
• Update Customer's Consents (Client): PUT /customers/{id}/consents - Docs

I've received a few errors originating from this line https://github.com/voucherifyio/voucherify-ruby-sdk/blob/master/lib/voucherify/client.rb#L145

unexpected token at ''
/usr/local/bundle/gems/json-2.6.3/lib/json/common.rb:216:in `parse',
/usr/local/bundle/gems/json-2.6.3/lib/json/common.rb:216:in `parse',
/usr/local/bundle/gems/voucherify-4.1.0/lib/voucherify/client.rb:145:in `initialize',
/usr/local/bundle/gems/voucherify-4.1.0/lib/voucherify/client.rb:93:in `new',
...

It appears that the error response is empty and the client cannot handle empty error responses. I've not be able to discover what is causing the empty error response. I can't find the request in the Voucherify API audit logs, so I suspect the request doesn't reach Voucherify. But it will help if the client can gracefully handle errors with empty responses.

I'm currently unable to cancel an order using the Ruby client. Cancelling orders using Curl works as expected, but making the request with the same request body using Ruby client doesn't work. Here are a couple of log ids

• log_0d446e83efc39dca2f (using Curl)
• log_0d44702edb9151d061 (using the Ruby client)

Since the requests are the same, I suspect the API is responding differently based on the user-agent or channel headers?

The Voucherify gem gives this warning under Ruby 2.7

lib/voucherify/service/vouchers.rb:19: warning: URI.escape is obsolete

According to the docs URI.encode is an alias for URI.escape, and

This method is obsolete and should not be used. Instead, use CGI.escape, URI.encode_www_form or URI.encode_www_form_component depending on your specific use case.

Hi there,
First of all I'd like to thank you for the work done at Voucherify, we've used for a few years and it's been a frictionless experience so far 🙂

I'm creating this issue because I have been working on migrating our voucherify gem from the legacy version to the new one.
I have succeeded in replacing every API call we had in our app except for one : creating a validation rule assignment (https://docs.voucherify.io/reference/create-validation-rule-assignment)

I've contacted support and Johanna advised me to create an issue here. I'll be happy to look into a code implementation if needed !

Thank you guys in advance

Add support for new methods related to Loyalty Programs.

Validation namespace:

• Validate Validation Rules: POST /validation-rules/:rule-id/validation - Docs

Rewards namespace:

• List: GET /rewards - Docs
• Create: POST /rewards - Docs
• Get: GET /rewards/:rewardId - Docs
• Update: PUT /rewards/:rewardId - Docs
• Delete: DELETE /rewards/:rewardId - Docs
• List Assignments: GET /rewards/:rewardId/assignments - Docs
• Create Assignment: POST /rewards/:rewardId/assignments - Docs
• Update Assignment: PUT /rewards/:rewardId/assignments/:assignmentId - Docs
• Delete Assignment: DELETE /rewards/:rewardId/assignments/:assignmentId - Docs

Loyalties namespace:

• List: GET /loyalties - Docs
• Create: POST /loyalties - Docs
• Get: GET /loyalties/:campaignId - Docs
• Update: PUT /loyalties/:campaignId - Docs
• Delete: DELETE /loyalties/:campaignId - Docs
• List Reward Assignments: GET /loyalties/:campaignId/rewards - Docs
• Create Reward Assignment: POST /loyalties/:campaignId/rewards - Docs
• Update Reward Assignment: PUT /loyalties/:campaignId/rewards/:assignmentId - Docs
• Delete Reward Assignment: DELETE /loyalties/:campaignId/rewards/:assignmentId - Docs
• List Earning Rules: GET /loyalties/{campaignId}/earning-rules - Docs
• Create Earning Rules: POST /loyalties/:campaignId/earning-rules - Docs
• Update Earning Rules: PUT /loyalties/:campaignId/earning-rules/:earningRuleId - Docs
• Delete Earning Rules: DELETE /loyalties/:campaignId/earning-rules/:earningRuleId - Docs
• List Members: GET /loyalties/:campaignId/members - Docs
• Add Member: POST /loyalties/:campaignId/members - Docs
• Get Member: GET /loyalties/:campaignId/members/:memberId - Docs
• Add Points: POST /loyalties/:campaignId/members/:memberId/balance - Docs
• Redeem Reward: POST /loyalties/:campaignId/members/:memberId/redemption - Docs

https://github.com/voucherifyio/voucherify-ruby-sdk/blob/master/lib/voucherify/service/distributions.rb#L20

We would like to reduce the max timeout from the default 60 seconds but the RestClient configuration doesn't seem to be accessible without monkey patching the client.

Would it be possible to poke through the timeout as a top-level config?

Thanks!

Hi,
Can't see utils functions like in that implementations:
https://github.com/rspective/voucherify.js/blob/master/lib/voucherify.js#L210
or
https://github.com/rspective/voucherify-nodejs-sdk/blob/master/utils.js

do I need to implement it for my ruby backend client?

The Voucherify documentation describes an endpoint to "Examine Qualification" that is not present in the Ruby SDK.

The Ruby SDK should expose this behavior through a create_qualifications_request method that takes a customer_id and optional context params, returning a list of campaigns said Customer is eligible for.