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

Gabriel/Step two #44

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Gabriel/Step two #44

wants to merge 3 commits into from

Conversation

grrlopes
Copy link

No description provided.

@grrlopes grrlopes closed this Jan 18, 2023
@grrlopes grrlopes reopened this Jan 18, 2023
@grrlopes grrlopes closed this Jan 18, 2023
@grrlopes grrlopes reopened this Jan 18, 2023
"description": "A guide to help developers onboard to the Button stack by building a simple todo app.",
"main": "index.js",
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1"
Copy link
Contributor

Choose a reason for hiding this comment

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

you need a script here to point postgraphile to your database & allow you to view the graphiql interface we looked at this morning.
The quickstart command here: https://www.graphile.org/postgraphile/introduction/ should be similar to what you need.
If it's working correctly, you should be able to go to the /graphiql endpoint and see the interface.

Copy link
Author

@grrlopes grrlopes Jan 24, 2023

Choose a reason for hiding this comment

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

I have added it into script at the step 4.
The Interface is running fine.

Copy link
Contributor

@dleard dleard left a comment

Choose a reason for hiding this comment

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

The package.json is missing the command that tells postgraphile where your database lives and to allow you to view the graphiql endpoint

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