Skip to content

fix : enable first-time setup wizard on main production build - #314

Open
mohammadsherif0 wants to merge 2 commits into
devfrom
feat-302-make_build-wizard-skip
Open

fix : enable first-time setup wizard on main production build#314
mohammadsherif0 wants to merge 2 commits into
devfrom
feat-302-make_build-wizard-skip

Conversation

@mohammadsherif0

Copy link
Copy Markdown
Collaborator

Don't skip wizard for make ENV=main build

Comment thread .env.main Outdated
Comment thread Dockerfile
RUN make ENV=$ENV build-frontend

CMD ["sh", "-c", "make ENV=$ENV NODE_ENV=production init dev-backend"]
CMD ["sh", "-c", "make ENV=$ENV NODE_ENV=production init dev-backend-wizard"]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to make sure, it is only used in the first run until settings are set?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes the wizard only runs while app.setup.wizardCompleted is unset/false. After setup completes, that setting is stored in the DB and the app routes to login/normal use instead of /wizard.

I also tested it locally and it works as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants