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 useUTC to types #690

Merged
merged 2 commits into from
Oct 19, 2019
Merged

Add useUTC to types #690

merged 2 commits into from
Oct 19, 2019

Conversation

eknowlton
Copy link
Contributor

@eknowlton eknowlton commented Sep 20, 2019

After some messing around with Nivo and line charts, I had a issue where I discovered the dates I was passing in were being converted to the clients timezone, even though I wasn't supplying a timezone with the date and these dates I was passing in were actually already correct. To get around this we discovered a useUTC option which we set to false.

Although it seems like its missing from the types.

After some messing around with Nivo and line charts, I had a issue where I discovered the dates I was passing in were being converted to the clients timezone, even though I wasn't supplying a timezone with the date and these dates I was passing in were actually already correct. To get around this we discovered a `useUTC` option which we set to false. 

Although it seems like its missing from the types.
Copy link
Contributor

@wyze wyze left a comment

Choose a reason for hiding this comment

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

LGTM.

packages/scales/index.d.ts Outdated Show resolved Hide resolved
@plouc plouc merged commit d7c1da5 into plouc:master Oct 19, 2019
@plouc
Copy link
Owner

plouc commented Oct 19, 2019

Thank you @eknowlton!

@eknowlton
Copy link
Contributor Author

Thank you for such a awesome library!

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