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

V2 APIs #5

Merged
merged 4 commits into from
Jun 13, 2018
Merged

V2 APIs #5

merged 4 commits into from
Jun 13, 2018

Conversation

MrBaummann
Copy link
Contributor

Heyho,

I added the available v2 APIs to the gem.
Will do Auth tomorrow :)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d50d5d5 on Downi:master into 7bb7c02 on parix:master.

@gottfrois
Copy link

+1 on this :)

@MrBaummann
Copy link
Contributor Author

Sadly I got no time for the Auth API at the moment..But I will do it, someday :P

@gottfrois
Copy link

i was only interested by the commerce endpoint anyway :)

@@ -0,0 +1,16 @@
module GW2
module Commerce
module Listings

Choose a reason for hiding this comment

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

should be Prices :)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling ce040f7 on Downi:master into 7bb7c02 on parix:master.

@parix
Copy link
Owner

parix commented Jun 1, 2015

This looks good, some tests are broken but that's ok I'll fix them.

@parix
Copy link
Owner

parix commented Jun 1, 2015

I'll be merging this tonight.

@MrBaummann
Copy link
Contributor Author

Yeah sorry about the tests I forgot about them the time I did the new tp listing stuff :D

@MrBaummann
Copy link
Contributor Author

I'll push the 'final' login key stuff tomorrow btw, if you want to wait for it :)

@parix
Copy link
Owner

parix commented Jun 3, 2015

Ah ok, I'll wait for it. =)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9c26133 on Downi:master into 7bb7c02 on parix:master.

@cbortz
Copy link

cbortz commented Aug 7, 2016

Thinking about experimenting with the API. Curious if there is any chance of this getting merged in?

@jzisser9
Copy link

jzisser9 commented May 8, 2018

Same - are there plans to move this forward? Anything I can do to help?

@parix parix merged commit 1433fda into parix:master Jun 13, 2018
@parix parix mentioned this pull request Jun 13, 2018
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.

None yet

6 participants