-
Notifications
You must be signed in to change notification settings - Fork 18
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
GitHub API Integration #8
Comments
I have the org repos being pulled into a list of 'repo' objects. It can be passed in a stream or future, whatever we want. |
@joeyda3rd Cool. Check out the It's a little tricky naming now due to github "repository" name, but that corresponds to "project" |
If that github package is being used and it returns some internal object and not JSON, some modifications will be needed, like adding a new constructor to |
Ya, it's being used and it does return an object but it literally took me 5 minutes. I can just as easily return a JSON. In fact, here's the response. https://api.github.com/orgs/Flutter-Buddies/repos |
that's nice. Are you working on this or can I just plug that in now? |
Go ahead and plug it in since you've got it set up. There's a little bit of scrubbing to do to "name" to remove dashes. I was just going to use String.replaceAll(). Description is fine as is. Not sure what to do about getting an image. Create an image file in the repo with a standard name? For the ongoing tag, maybe we can use either the 'archived' or 'disabled', boolean? |
Ok. I'll now just plug it in with basic data and then you and @Zambrella can play around with which else data we want to show in app, as there is really a lot of cool data. Then you'll be just improving on |
Ok, now it's using real API. I left the image to be random and didn't format name. You can do that. I'm gonna step off this issue now for a while, at least implementation-wise. |
Need to catch null response fields. |
No description provided.
The text was updated successfully, but these errors were encountered: