-
Notifications
You must be signed in to change notification settings - Fork 2
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
Switch to new Unibeautify CI Playground API #26
Conversation
The maintainers of this repository would appreciate it if you could provide more information. |
src/Container.tsx
Outdated
@@ -14,7 +14,7 @@ require("react-github-button/assets/style.css"); | |||
import ApiClient, { SupportResponse } from "./ApiClient"; | |||
import { Playground } from "./Playground"; | |||
|
|||
const apiUrl: string = "https://unibeautify-playground-eijdodv3e08v.runkit.sh"; | |||
const apiUrl: string = "http://ntd6xp2n56.execute-api.us-east-1.amazonaws.com/dev/playground"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No https
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good eye! 👀 I was testing to see if it would work locally with http
since the localhost server is only http
. No dice. Then forgot to fix before pushing 🙄.
Probably should review the Node versions Travis builds against. Right now it's using 6 and 10. I would suggest switching 6 to 8 or just adding 8 in there. |
Having problems with https://5b0c8b9db13fb1142da4a881--unibeautify-playground.netlify.com/ Chrome shows in Network request's preview: “Failed to load response data” |
@szeck87 Ready for testing 😃 |
No description provided.