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

Payment date should be in EST TimeZone #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dima4p
Copy link

@dima4p dima4p commented Jun 1, 2015

Additionally we allow the string to be sent to.

payment_date: Time.current, currency: 'USD')
unless payment_date.is_a? String
payment_date = payment_date.in_time_zone('EST').strftime('%m/%d/%Y %H:%M:%S')
end

Choose a reason for hiding this comment

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

does their API really take timestamps in EST? 😵

Copy link
Author

Choose a reason for hiding this comment

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

I have document named Payoneer_integration_guide.pdf that states
p9 (HTTP API)
Payment Date

[Required] Date of payment (EST), Normally, the
specified date is ignored, and the payment is executed
immediately upon payment submission.
A partner can be configured in the Payoneer system
for “Payments scheduling”. Using this configuration
(which is inactive by default), the payment is executed
only at the time specified by the “Payment date”
parameter.

Numeric
(MM/dd/yyyyHH:mm:ss)

Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch on that. It is weird that Payoneer would take EST, specifically. I wonder if the docs mean to say that they expect US Eastern time (EST or EDT).

@troq would you be able to send them an email for verification on this? Is it supposed to be EST or do they acknowledge daylight saving time? Or are they just tricking us and they're really expecting UTC?

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.

3 participants