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

Profile, SC2, and WoW have all been moved to their own packages #7

Closed
wants to merge 20 commits into from

Conversation

nmccrory
Copy link
Contributor

Just a heads up this is a nice sized PR... I moved Account() and Profile() services into their own package go-bnet/profile. All sc2 and wow prefixed structs have been moved into their own respective packages as well. Changes have also been made to client.go to make the HTTP helper functions easier to reuse.

I also made a few small changes to README.md to keep it up to date with the project.

@nmccrory
Copy link
Contributor Author

@mitchellh please take a look at this when you get a chance. Would love to keep building out the library (:

@nmccrory nmccrory closed this Jun 25, 2017
@nmccrory nmccrory reopened this Jun 25, 2017
removed unused imports inside of client.go
Cleaning up workspace to begin development from nmccrory/go-bnet fork
…lso noticed a mistake in the client.go file of the go-bnet package... There was a need for a NewClient method for use in the client_test setup().
Created a close to working service for profiles in the sc2 package. A…
…d the Matches() method to retrieve a Starcraft 2 profile's match history.
…rom github.com/mitchellh/go-bnet/.. to make merging easier.
@nmccrory nmccrory closed this Jul 7, 2017
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.

1 participant