-
Notifications
You must be signed in to change notification settings - Fork 969
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
fix: faq #2101
fix: faq #2101
Conversation
Thank you, this looks great! The CI is failing because some files are formatted incorrectly. To format them, run:
Thank you! |
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.
forgot to click on submit... maybe you can open a follow up PR?
If multiple | ||
[registration methods](../docs/self-service/flows/user-registration.mdx) are | ||
configured, users may be redirected to a social login provider | ||
to link the accounts after the email verification. To skip this link-up add the | ||
session hook after the registration method (e.g. `password`): |
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.
Not sure that statement is true?
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.
The session hook will not skip the link up?
At least for the user who asked this, it solved the issue.
Should I revert this and test it before we add it?
It also seems like an edgecase...
Thanks a lot for the review @zepatrik, I will open a follow up PR. |
This patch - moves the FAQ to the Debug & Help section - renames it to Tips & Troubleshooting - moves many of the questions to documents where they fit better, reformatted and with added information where needed. - also some other spelling/format fixes See also ory/docusaurus-template#87
The FAQ currently is outdated, see questions on SPAs and others.
This patch
PR in docusaurus-template to remove the FAQ there:
ory/docusaurus-template#87
Related issue(s)
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security. vulnerability, I
confirm that I got green light (please contact
[email protected]) from the maintainers to push
the changes.
works.
Further Comments