-
Notifications
You must be signed in to change notification settings - Fork 535
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
Change shell scripts to run new UI #533
Change shell scripts to run new UI #533
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested. Reviewed everything up to 6dbf443 in 31 seconds
More details
- Looked at
70
lines of code in3
files - Skipped
1
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_6UKRswBas6hN7liB
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
@@ -127,7 +128,7 @@ Note: Our setup script does these two for you, but they are here for reference. | |||
```bash | |||
./run_ui.sh | |||
``` | |||
1. Navigate to `http://localhost:8501` in your browser to start using the UI | |||
1. Navigate to `http://localhost:8080` in your browser to start using the UI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is 8051 for streamlit and 8080 for our new UI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
Summary:
Updated shell scripts to support new UI setup and configure frontend environment.
Key points:
run_ui.sh
to navigate toskyvern-frontend
and runnpm start
.setup.sh
to ensurenpm
is installed and to install frontend dependencies.setup.sh
to configureskyvern-frontend/.env
with the API token.skyvern-frontend/.env.example
to includeVITE_SKYVERN_API_KEY
placeholder.Generated with ❤️ by ellipsis.dev