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.

Sign up for GitHub

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

Jump to bottom

Bfe 178 ipfs fleek ci workflow #42

Merged
merged 63 commits into from
Sep 6, 2024
Merged

Bfe 178 ipfs fleek ci workflow #42

merged 63 commits into from
Sep 6, 2024

Conversation

bearpong
Copy link
Contributor

@bearpong bearpong commented Aug 30, 2024 •
edited
Loading

This is on top of #30. Please merge that PR first.

this PR adds:

  • ci static build on pr to check it doesn't throw errors
  • fleek deploy when main branch is updated
  • .env.bartio file and pnpm setenv:bartio script
  • .env.artio even though a lot of variables are missing on artio. I think it should be declared as not supported anymore.

All domains under *.berachain.com have been whitelisted on Dynamic to solve cors issues.

Plese after merging this PR, all developers should:

  1. move secrets from .env to .env.local
  2. run pnpm setevn:bartio
  3. stop editing .env and keep .env.bartio updated
  4. Updated vercel build settings to use .env.bartio and keep only secrets on the vercel environment

bearpong added 30 commits August 20, 2024 10:24
@bearpong
BFE-56 moving berps to ipfs
de8443b
@bearpong
minor clean up
88f1837
@bearpong
chore: add .env.bartio to git
f7e4e6c
@bearpong
chore: set-env to copy env presets to .env
3349887
@bearpong
ci: setup Test static build on IPFS Refs: BFE-178
ac3648c
@bearpong
fix: pnpm version conflict with package.json BFE-178
e088d26
@bearpong
ci: deploy on fleek Refs: BFE-178
237df4a
@bearpong
build: set env files during ci workflow
1b18d3e
@bearpong
ci: updated deploy on fleek
ffd26a2
@bearpong
build: update flows
648a127
@bearpong
revert: "chore: set-env to copy env presets to .env" to restore index…
f1c3552 
….tsx file in charts

This reverts commit 3349887.
@bearpong
build: update flows env
9815d48
@bearpong
ci: cache
a7ad490
@bearpong
build: dotenv -c param to include .env.local
63371a9
@bearpong
revert: removed cache in workflows
9dcb85d
@bearpong
ci: add env to actions
e4bcbf4
@bearpong
fix: typo
1952c51
@bearpong
fix: minor
692e1b3
@bearpong
ci: assign environment
5ff7b9e
@bearpong
ci: added repo secrets instead of env
59d5fc1
@bearpong
fix: wrong .env.bartio
54518df
@bearpong
build: added env vars
3899537
@bearpong
fix: adding env to build cmd
47a965b
@bearpong
build: updated vars
d7e8eba
@bearpong
ci: added vars
4bf0e58
@bearpong
ci: trying to fix build
dc49f88
@bearpong
ci: debugging fail
38edf1f
@bearpong
fix(perp): start trading on ipfs
a749dcc
@bearpong
build(perp): added build:ipfs to avoid wrongful builds
550ae95
@bearpong
fix(perp): minor
f680b86
@bearpong bearpong requested a review from ninibear030 September 5, 2024 15:32
@ninibear030
Copy link
Contributor

This is on top of #30. Please merge that PR first.

this PR adds:

  • ci static build on pr to check it doesn't throw errors
  • fleek deploy when main branch is updated
  • .env.bartio file and pnpm setenv:bartio script
  • .env.artio even though a lot of variables are missing on artio. I think it should be declared as not supported anymore.

In order to have this work properly, we should whitelist in Dynamic settings:

  • ipfs.bartio.bend.berachain.com
  • ipfs.bartio.berps.berachain.com
  • ipfs.bartio.bex.berachain.com
  • ipfs.bartio.honey.berachain.com
  • hub?

However, we can't whitelist all CID domains like XXXX.ipfs.dweb.link. We might use a wildcard, but all websites on IPFS might use our API key... need to find a solution to that.

Plese after merging this PR, all developers should:

  1. move secrets from .env to .env.local
  2. run pnpm setevn:bartio
  3. stop editing .env and keep .env.bartio updated
  4. Updated vercel build settings to use .env.bartio and keep only secrets on the vercel environment

should be these 4

  • ipfs.bartio.bend.berachain.com
  • ipfs.bartio.berps.berachain.com
  • ipfs.bartio.hub.berachain.com
  • ipfs.bartio.honey.berachain.com

ninibear030
ninibear030 approved these changes Sep 5, 2024
View reviewed changes
apps/honey/fleek.config.json Outdated
{
"sites": [
{
"slug": "bartio-honey",
Copy link
Contributor

@ninibear030 ninibear030 Sep 5, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

v2 ^

apps/perp/fleek.config.json Outdated
{
"sites": [
{
"slug": "berps-bartio",
Copy link
Contributor

@ninibear030 ninibear030 Sep 5, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

v2-berps^

ninibear030 and others added 3 commits September 6, 2024 11:06
@bearpong bearpong merged commit 9c7040f into v2 Sep 6, 2024
19 of 21 checks passed
@bearpong bearpong deleted the BFE-178-ipfs-fleek-ci-workflow branch September 6, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ninibear030 ninibear030 ninibear030 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bearpong @ninibear030