Closed florlafuente closed 6 years ago
@florlafuente I have my doubts about using /settings
as the name for the route. Might be better to use.. init-settings
or something similar? Maybe we should find an example
Also, I see that there is already a control that if settings are already initialized, it shows a error message "initialized". Shouldn't it just redirect you to the home page? Like.. its a one-time route. Thoughts?
@guillecro Ok with changing page name. It was /settings-init and then I changed to /settings. I'll make the roll back. With the error message subject, I don't know... I think your opinion makes sense but I'm not so sure...
Creates page /settings with a form that sets basic setting config: communityName and mainColor. If skip button is pressed, sets default values. Then, in both cases, redirects to /admin.
Also, this branch adds a color picker input in admin/settings to change mainColor.