-
Notifications
You must be signed in to change notification settings - Fork 29.9k
Add a section in contributing guide describing owner user creation and reset #16006
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
base: master
Are you sure you want to change the base?
Conversation
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.
cubic reviewed 1 file and found no issues. Review PR in cubic.dev.
CONTRIBUTING.md
Outdated
If for any reason the credentials are lost before setting up password recovery, you can either remove the database file or run the following SQL command to reset the owner user: | ||
|
||
``` | ||
update settings SET value='false' where key='userManagement.isInstanceOwnerSetUp'; |
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.
We would recommend using the server command over directly editing the database: https://docs.n8n.io/hosting/cli-commands/#user-management
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.
Replaced sql with ./packages/cli/bin/n8n user-management:reset
@@ -169,6 +169,21 @@ To start n8n with tunnel: | |||
./packages/cli/bin/n8n start --tunnel | |||
``` | |||
|
|||
## First run | |||
|
|||
The first time you run n8n locally you will be prompted to create an owner user. |
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.
Is this information really needed? it is the same as normally running n8n so this shouldn't be a surprise.
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.
It's definitely not a suprise during first run. It still adds some context for users that do need to reset the credentials (e.g. people that tried running n8n locally once or twice and return back to it a couple of weeks later). So, I'd say let's keep it but I am perfectly fine with removing it.
Hey @iocanel, Thanks for the PR, We have created "GHC-2341" as the internal reference to get this reviewed. One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team. |
Hey @iocanel, Thanks for the PR, We have created "GHC-2342" as the internal reference to get this reviewed. One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team. |
2de44e0
to
071b346
Compare
@iocanel I will run this past the docs team and see what they think. |
Summary
The pull request introduces a section in the CONTRIBUTING.md that explain the process of owner user creation and reset.
Related Linear tickets, Github issues, and Community forum posts
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)