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

Update README.md root endpoint #84

Closed
wants to merge 1 commit into from

Conversation

clemon
Copy link

@clemon clemon commented Apr 11, 2018

.io -> .com
took me a few minutes to figure out why my api calls weren't working :)

.io -> .com
@azenMatt azenMatt requested a review from pnavarrc April 11, 2018 19:02
@clemon
Copy link
Author

clemon commented Apr 11, 2018

nvm, I also put app instead of api, curled the html and assumed success

@clemon clemon closed this Apr 11, 2018
@clemon clemon deleted the clemon-update-root-endpoint branch April 11, 2018 19:08
@pnavarrc
Copy link
Contributor

pnavarrc commented Apr 12, 2018

Hi @clemon! We haven't moved the API endpoints to .com yet (public or private) because that will impact the permissions on the extension and we don't really need to change it.

Still, thanks for paying attention and for submitting the pull request! 🎉

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.

2 participants