-
Notifications
You must be signed in to change notification settings - Fork 62
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
User journey #98
Comments
Hello, I saw issue #94 a few days ago and saw you the "user journey" but im not familiar with that concept. Would you consider to allow logins regardless of registration being disabled or add another setting to control this? |
probably would be best to have an option to create an admin account with env variables, and disable upload for non admins. I don't want random people uploading shit to my server. |
@luismanson / @dada513 Yeah, it would be an admin dashboard where you can adjust the settings you would like for your application. That is my thoughts on this as well. So the admin allows either anyone to register an account. Organizations. Add people manually. Or something in between here. @dada513 Sure, feel free to create LDAP support. I think more people would have the need of that as well |
@bjarneo I think for settings it's alright to do it like other services and just have configuration via environment variables, like HEMMLIG_ALLOW_REGISTER, makes deployment easier to be automated. But an admin panel would be useful for creating users though |
Where would these settings be stored? Plain files? Database? Would it automatically alter default.cjs or production.cjs? Add an additional file? Using env variables bloats the systemd service file and may allow unix users on host to view the variable content where a configuration file or store in database is easier to restrict. |
It will be saved to the database whenever that part is ready. |
The text was updated successfully, but these errors were encountered: