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 /user endpoint support #90

Merged
merged 1 commit into from
Oct 15, 2017
Merged

Add /user endpoint support #90

merged 1 commit into from
Oct 15, 2017

Conversation

dpc
Copy link
Contributor

@dpc dpc commented Oct 15, 2017

No description provided.

@dpc
Copy link
Contributor Author

dpc commented Oct 15, 2017

Just hacked this together. I need to actually test in my app, but feel free to point out any wrongdoings already. :)

@softprops
Copy link
Owner

Well that was fast!

@softprops softprops merged commit fada67b into softprops:master Oct 15, 2017
@softprops
Copy link
Owner

I'll publish a new release tomorrow. Thanks for the pull @dpc!

@dpc
Copy link
Contributor Author

dpc commented Oct 15, 2017

I don't know why but it's not working for me right now. I've replaced my reqwest code doing it with this one, and I'm geting: invalid type: null, expected a string at line 1 column 852! . Investigating.

@softprops
Copy link
Owner

That typically means one of the String fields needs to be of Option type

@dpc
Copy link
Contributor Author

dpc commented Oct 15, 2017

Oh. It's hireable. I don't even know what is it's actual type, since it's "null" for me.

@softprops
Copy link
Owner

published a new version with your changes https://crates.io/crates/hubcaps and added some examples

@dpc
Copy link
Contributor Author

dpc commented Oct 16, 2017

Thanks!

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