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

Fix json unmarshal from json api to Task struct #1

Merged
merged 2 commits into from
Oct 10, 2019

Conversation

massawho
Copy link

@massawho massawho commented Oct 9, 2019

#Hacktoberfest

The json from the api returns a date format like "2019-10-07T14:00:00Z"
and this makes the code to panic. This is because the CustomDate is
expecting a timestamp with the format "2006-01-02T15:04:05+00:00".

wtfutil/wtf#645

The json from the api returns a date format like "2019-10-07T14:00:00Z"
and this makes the code to panic. This is because the CustomDate is
expecting a timestamp with the format "2006-01-02T15:04:05+00:00".

wtfutil/wtf#645
@senorprogrammer
Copy link

Sweet. I love it when a PR deletes code.

The json from the api returns a date format like "2019-10-07T14:00:00Z"
and this makes the code to panic. This is because the CustomDate is
expecting a timestamp with the format "2006-01-02T15:04:05+00:00".

wtfutil/wtf#645
@senorprogrammer senorprogrammer merged commit 56902a6 into wtfutil:master Oct 10, 2019
@senorprogrammer
Copy link

Works nicely, thanks!

@massawho
Copy link
Author

sweet! i'm glad it was merged :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants