jayden-chan / puggies

Puggies is a self-hosted CS:GO demo analyzer and statistics platform
GNU Affero General Public License v3.0
3 stars 2 forks source link

Doesn't get any initial setup wizard - update initial install instructions? #61

Open NorskNoobing opened 2 months ago

NorskNoobing commented 2 months ago

I don't get the setup wizard on first boot, like what's mentioned on the configuration page:

the only account will be the admin account which you must set up when first installing Puggies. Accounts can still be created manually by the admin user.

https://github.com/NorskNoobing/puggies/blob/master/docs/Configuration.md#puggies_allow_self_signup

This can be circumvented by temporarily setting the self-signup env-var to true:

  puggies:
    environment:
      - PUGGIES_ALLOW_SELF_SIGNUP=true

I'm unsure of what the initial setup wizard does, other than creating the admin user, and what implications it might have when it's not being run. I'd like to see either the documentation updated if the setup wizard is deprecated, or try to fix the setup wizard so it shows up on first boot.

After testing with self-registration, it seems that the first user that's created via self-signup is always assigned the ADMIN role, and any signups after that has no automatic role assignment. There should be more info on the initial setup page, that would help users through the process of temporarily enabling user registration, then optionally disabling it again.

This leads me to believe that only the documentation has to be updated. Please correct me if I'm wrong :)

NorskNoobing commented 2 months ago

Additionally when using the image building instructions:

git clone https://github.com/jayden-chan/puggies.git
cd puggies
./docker-build.sh

it won't produce an image named jayden-chan/puggies:${TAG}, as this is only when the image is publically listed on Dockerhub. The image is just named puggies or puggies:latest by default. The composefile docs should be updated to just have the image name of puggies:latest when fixing the other issue listed in the post above.