-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: Add polygon amoy support #10
Conversation
const data = (await response.json()) as T | ||
return data | ||
} catch (error) { | ||
console.error(error) |
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.
What do you think about propagating the error when it occurs?
src/lib/environment.ts
Outdated
} | ||
|
||
return isPrd || isStg ? 1 : 11155111; | ||
})(); | ||
|
||
export const peerUrl = (() => { | ||
if (process.env.REACT_APP_PEER_URL) { | ||
return process.env.REACT_APP_PEER_URL; | ||
if (process.env.VITE_REACT_APP_PEER_URL) { |
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.
This won't be ever set, as the process.env.VITE_REACT_APP_PEER_URL
must be defined in the vite config because vite does not use the process.env
constant.
The same will happen with other environment variables. What do you think about adding the environments module as we have in other dApps?
ce9eb94
to
6c4bf9c
Compare
fa197a1
to
0e7fed7
Compare
.github/workflows/set-rollouts.yaml
Outdated
@@ -24,13 +24,14 @@ jobs: | |||
packageVersion: ${{ github.event.deployment.payload.packageVersion }} | |||
|
|||
# Rollout information | |||
deploymentDomain: "third-party-aggregator.decentraland.zone" | |||
deploymentName: "_site" | |||
deploymentPath: 'third-party-aggregator' |
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.
We're planning to modify the site a bit and make it more user friendly. What do you think about using the /linked-wearables
path here?
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.
sure, it's a great idea! updated :)
No description provided.