Skip to content

build: fix bee hosts for PR preview#253

Merged
vojtechsimetka merged 2 commits intomasterfrom
build/fix-pr-preview
May 10, 2022
Merged

build: fix bee hosts for PR preview#253
vojtechsimetka merged 2 commits intomasterfrom
build/fix-pr-preview

Conversation

@agazso
Copy link
Copy Markdown
Contributor

@agazso agazso commented Apr 22, 2022

The REACT_APP_BEE_HOSTS environment variable was set to use the old way of addressing gateway modes that is used for the PR previews. This PR fixes it to use the correct url.

@bee-worker
Copy link
Copy Markdown
Collaborator

bee-worker commented Apr 22, 2022

🐝 PR preview in Swarm

Preview URL: https://bah5acgzat64kznzgrjz5cnkuaeknpnr3sd7tx65pz4yujtly37fibmgupewa.bzz.link
Swarm Hash: 9fb8acb7268a73d135540114d7b63b90ff3bfbafcf3144cd78dfca80b0d4792c
Commit Hash: 2300705
Commit Message: chore: update .github/workflows/check.yaml

Comment thread .github/workflows/check.yaml Outdated

env:
REACT_APP_BEE_HOSTS: 'https://bee-0.gateway.ethswarm.org/,https://bee-1.gateway.ethswarm.org/,https://bee-2.gateway.ethswarm.org/,https://bee-3.gateway.ethswarm.org/,https://bee-4.gateway.ethswarm.org/,https://bee-5.gateway.ethswarm.org/,https://bee-6.gateway.ethswarm.org/,https://bee-7.gateway.ethswarm.org/,https://bee-8.gateway.ethswarm.org/,https://bee-9.gateway.ethswarm.org/'
REACT_APP_BEE_HOSTS: 'https://api.gateway.ethswarm.org/'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These should still be exposed, not sure why it does not work but will check with devops. Using the api.gateway.ethswarm means we lose the frontend optimization. See #218

Comment thread .github/workflows/check.yaml Outdated
@vojtechsimetka vojtechsimetka merged commit 254f008 into master May 10, 2022
@vojtechsimetka vojtechsimetka deleted the build/fix-pr-preview branch May 10, 2022 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants