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

attempt bug fix: You have reached the rate limit for trading_times #575

Merged
merged 3 commits into from
Nov 1, 2018

Conversation

bruce-binary
Copy link
Contributor

the now time should be using local date not UTC

bruce-binary and others added 3 commits October 30, 2018 14:17
the now time should be using local date not UTC
…date will be the current date, but smartcharts will also request trading times for the next day
@bruce-binary bruce-binary force-pushed the markets-undefined-fix branch from 60dcd98 to 8e434d1 Compare October 31, 2018 03:44
Copy link
Contributor

@shayan-binary shayan-binary left a comment

Choose a reason for hiding this comment

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

LGTM

@bruce-binary bruce-binary merged commit bdea1e5 into deriv-com:dev Nov 1, 2018
@bruce-binary bruce-binary deleted the markets-undefined-fix branch November 1, 2018 05:35
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.

2 participants