-
Notifications
You must be signed in to change notification settings - Fork 435
fix: correct Auth0 export users script and update documentation #2330
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: correct Auth0 export users script and update documentation #2330
Conversation
vinckr
left a comment
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.
Nice!
looks pretty good already, I would do a test of the script and left some comments on style.
I think the numbered list here don't add anything and in general would be better without.
there are 2 steps only anyway so its not that hard to navigate and just looks confusing in the ToC (see screenshot below).
So please remove the Phase X parts and numbered list in headings.
also in line with the style guide: "Avoid using symbols and special characters in headings."
![]()
e223ec6 to
fb8ed63
Compare
unatasha8
left a comment
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.
Added a bunch of suggestions.
Related Issue or Design Document
Fixes the Auth0 export script and improves the migration guide documentation for better clarity.
Changes
Script Fixes (
0-get-auth0-user-data.sh)Documentation Improvements (
auth0.mdx)AUTH0_USERDATA.jsonautomatically.Impact
Checklist
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.