Skip to content
This repository has been archived by the owner on Dec 22, 2019. It is now read-only.

Suggestion add api()->get() example to readme #116

Open
ddeath opened this issue May 13, 2017 · 1 comment
Open

Suggestion add api()->get() example to readme #116

ddeath opened this issue May 13, 2017 · 1 comment

Comments

@ddeath
Copy link

ddeath commented May 13, 2017

Hi there,

I would like to suggest to add api()->get() request, which needs domain, to readme as example.
For example:

Mailgun::api()->get('my-domain/tags');

I was assuming that it will be added by library because I specified domain in config file. But it is not case.
Also for example call:

Mailgun::api()->get('/tags');

returns:

{#880
 +"http_response_body": "Mailgun Magnificent API",
 +"http_response_code": 200,
}

It took me some time to figure it out :) Maybe it will save some time to others :)

I know that it is my fault because in the mailgun api is written endpoint as 'domain/tags' and I made wrong implication that setting domain in config file is sufficient...

@newdygo
Copy link

newdygo commented Jun 21, 2017

Thanks man, you help a lot.

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

No branches or pull requests

2 participants