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

Use CORS compatible domain in example #282

Merged
merged 1 commit into from
Feb 18, 2016
Merged

Conversation

vabruzzo
Copy link

fixes example to use api.github.com rather than github.com

@mislav
Copy link
Contributor

mislav commented Feb 18, 2016

The example URL is just for demonstration purposes only. It doesn't really matter what the exact URL is. People are to substitute their own URL in this field. The URL github.com works well as an example because it's simple and also refers to the entity behind this project.

@mislav mislav closed this Feb 18, 2016
@vabruzzo
Copy link
Author

Shouldn't the example at least work if someone copies and pastes it?

@mislav mislav reopened this Feb 18, 2016
@mislav
Copy link
Contributor

mislav commented Feb 18, 2016

You're right! api.github.com allows for CORS and therefore will work when github.com doesn't. Thanks!

mislav added a commit that referenced this pull request Feb 18, 2016
@mislav mislav merged commit 31bd9c4 into JakeChampion:master Feb 18, 2016
@vabruzzo vabruzzo deleted the patch-1 branch February 19, 2016 00:12
@dgraham
Copy link
Contributor

dgraham commented Feb 19, 2016

/cc #280

@dgraham dgraham changed the title fix example Use CORS compatible domain in example Feb 19, 2016
@mislav
Copy link
Contributor

mislav commented Feb 19, 2016

@dgraham Ah, thanks for that bit of context. @vga- It would have been helpful to me if you linked to the prior discussion when opening the PR. More context around the proposed change always helps!

@vabruzzo
Copy link
Author

Okay, thanks! I'll make sure to do that in the future.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants