Skip to content
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

Prompt for template on reflex init #2122

Merged
merged 4 commits into from Nov 3, 2023
Merged

Prompt for template on reflex init #2122

merged 4 commits into from Nov 3, 2023

Conversation

picklelo
Copy link
Contributor

@picklelo picklelo commented Nov 2, 2023

Ask the user to specify a template interactively on the first reflex init.

@picklelo
Copy link
Contributor Author

picklelo commented Nov 2, 2023

One thing I haven't included yet - if the user specifies a template up front we should skip this check. Currently if they do reflex init --template blank it still goes through the prompt. Maybe we can remove the default argument in the init command to catch this.

Copy link
Collaborator

@masenf masenf left a comment

Choose a reason for hiding this comment

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

yeah should be easy enough to kill the default template on line 125, technically there is no default template now

Copy link
Collaborator

@masenf masenf left a comment

Choose a reason for hiding this comment

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

need to add --template blank to the docker integration test

masenf
masenf previously approved these changes Nov 3, 2023
@@ -17,7 +17,7 @@ RUN python3.11 -m venv $VIRTUAL_ENV
RUN pip install -r requirements.txt

# Deploy templates and prepare app
RUN reflex init
RUN reflex init --template blank
Copy link
Collaborator

Choose a reason for hiding this comment

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

we actually probably don't need this in the dockerfile, because the app should already exist when the container is being built... not sure if it hurts anything though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point - I'll remove this

@picklelo picklelo merged commit 6e1bce3 into main Nov 3, 2023
37 checks passed
@masenf masenf deleted the nikhil/template-cli branch December 6, 2023 22:18
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.

None yet

2 participants