这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@marionschleifer
Copy link
Contributor

@marionschleifer marionschleifer commented May 20, 2020

Description

This PR removes the incomplete troubleshooting guide from the docs.

It adds a temporary page that contains references as to where to find solutions on errors.

Later we'll replace this with a comprehensive page with all errors.

Changelog

  • CHANGELOG.md is updated with user-facing content relevant to this PR.

Affected components

  • Docs

Related Issues

#4052

Steps to test and verify

https://deploy-preview-4825--hasura-docs.netlify.app/graphql/manual/troubleshooting/index.html

@marionschleifer marionschleifer added the c/docs Related to docs label May 20, 2020
@marionschleifer marionschleifer requested review from a team as code owners May 20, 2020 14:05
@netlify
Copy link

netlify bot commented May 20, 2020

Deploy preview for hasura-docs ready!

Built with commit 02d36a9

https://deploy-preview-4825--hasura-docs.netlify.app

CHANGELOG.md Outdated
- cli: remove irrelevant flags from init command (close #4508) (#4549)
- docs: update migrations docs with config v2 (#4586)
- docs: update actions docs (#4586)
- docs: replace incomplete trouble shooting section with reference on debugging errors (close #4052) (#4825)
Copy link
Contributor

Choose a reason for hiding this comment

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

This appears to be in wrong place.

returned by the describe command as it is something added by Hasura GraphQL engine. Now if a new column is created
with the same name, i.e. ``author``, via ``psql``, the Hasura GraphQL engine will throw the above error when restarted as it has two
references to the ``author`` field for the ``article`` table.
If you didn't find a solution in any of the abovementioned sections or if you prefer to troubleshoot with the community,
Copy link
Member

Choose a reason for hiding this comment

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

Typo in: "above mentioned"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's actually a word: https://www.merriam-webster.com/dictionary/abovementioned :D but it's fine for me to change :)

Copy link
Member

Choose a reason for hiding this comment

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

Wow. TIL. In that case sorry for changing it :)

@rikinsk rikinsk merged commit 93bb342 into hasura:master May 21, 2020
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-4825.herokuapp.com is deleted

stevefan1999-personal pushed a commit to stevefan1999-personal/graphql-engine that referenced this pull request Sep 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/docs Related to docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants