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

fix(build): Fix yarn start error #853

Merged

Conversation

MortalHappiness
Copy link
Member

TL;DR

Fix the yarn start error in the issue.

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

Fix the yarn start error in the issue by reading from the environment variable instead of hardcoing for the variable LOCAL_DEV_HOST.

Tracking Issue

Resolves: flyteorg/flyte#5075

Follow-up issue

NA

Copy link

welcome bot commented Mar 19, 2024

Thank you for opening this pull request! 🙌

These tips will help get your PR across the finish line:

  • Most of the repos have a PR template; if not, fill it out to the best of your knowledge.
  • Sign off your commits (Reference: DCO Guide).

@austin362667
Copy link

austin362667 commented Mar 19, 2024

looks great, thanks a lot!

@MortalHappiness MortalHappiness force-pushed the bugfix/#5075-yarn-start-fail branch from 0bfecb8 to 127aee2 Compare March 19, 2024 20:03
@pingsutw pingsutw merged commit 3920ce8 into flyteorg:master Mar 27, 2024
7 checks passed
Copy link

welcome bot commented Mar 27, 2024

Congrats on merging your first pull request! 🎉

@flyte-bot
Copy link
Collaborator

🎉 This PR is included in version 1.11.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

jsonporter added a commit that referenced this pull request Apr 1, 2024
Signed-off-by: Jason Porter <[email protected]>
jsonporter added a commit that referenced this pull request Apr 2, 2024
* init commit for triggers UI

Signed-off-by: Jason Porter <[email protected]>

* Merged #853 back, removed log

Signed-off-by: Jason Porter <[email protected]>

* Fixed test

Signed-off-by: Jason Porter <[email protected]>

---------

Signed-off-by: Jason Porter <[email protected]>
jsonporter added a commit that referenced this pull request Apr 19, 2024
Signed-off-by: Jason Porter <[email protected]>
@jsonporter jsonporter mentioned this pull request Apr 19, 2024
8 tasks
MortalHappiness added a commit to MortalHappiness/flyteconsole that referenced this pull request Apr 19, 2024
Only set ADMIN_API_URL to localhost:30080 when NODE_ENV is development

Signed-off-by: Chi-Sheng Liu <[email protected]>
@MortalHappiness MortalHappiness mentioned this pull request Apr 19, 2024
8 tasks
MortalHappiness added a commit to MortalHappiness/flyteconsole that referenced this pull request Apr 19, 2024
jsonporter pushed a commit that referenced this pull request Apr 19, 2024
Signed-off-by: Chi-Sheng Liu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Flyte console cannot be started for local development
4 participants