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(site): set PUBLIC_API_BASE_URL during build #36

Merged
merged 2 commits into from
Sep 2, 2023
Merged

fix(site): set PUBLIC_API_BASE_URL during build #36

merged 2 commits into from
Sep 2, 2023

Conversation

veeti-k
Copy link
Collaborator

@veeti-k veeti-k commented Sep 2, 2023

No description provided.

Copy link
Contributor

@AriPerkkio AriPerkkio left a comment

Choose a reason for hiding this comment

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

Lisätään vielä packages/site/.env.local, jossa

PUBLIC_API_BASE_URL=http://localhost:3000/oulu-dev-meetups/api

... ja pitää myöhemmin laittaa vaikkapa msw rullaan Astron kaveriksi tuohon endpointtiin. Tai saatasko tuo packages/api lokaalissa ajoon msw + octokit-mockin kanssa?

@AriPerkkio AriPerkkio changed the title Pass worker url to frontend at build time fix(site): set PUBLIC_API_BASE_URL during build Sep 2, 2023
@veeti-k
Copy link
Collaborator Author

veeti-k commented Sep 2, 2023

packages/api lokaalissa ajoon msw + octokit-mockin kanssa kuulostaa kyllä hyvältä

@veeti-k veeti-k merged commit ca9b405 into PlanBee-dev:master Sep 2, 2023
1 check passed
@veeti-k veeti-k deleted the frontend-conf branch September 2, 2023 11:59
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