-
Notifications
You must be signed in to change notification settings - Fork 412
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
Addition of USERNAME and PASSWORD to environment variables to create Owner user. #5506
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.
Hi, this looks good. Can you also confirm this has been added to the right documentation in the argilla-server directory and potentially also argilla docs itself. https://docs.argilla.io/latest/reference/argilla-server/configuration/
and the readme for the server. Thanks for the contribution :)
|
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.
@bikash119, one more remark. After that, I will merge the PR :)
Co-authored-by: David Berenstein <[email protected]>
Commented the USERNAME and PASSWORD field. This will enforce users to set USERNAME and PASSWORD for Owner explicitly. Thank you @david for the review comments.
Included default username/password to environment variable to create owner user.
Description
Closes #5434
Type of change
How Has This Been Tested
In dev machine
Checklist