-
Notifications
You must be signed in to change notification settings - Fork 695
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
Remove unused env vars. update readme. Deploy your own frontend. #3041
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,17 +1,9 @@ | ||
# Required | ||
# Covalent (https://www.covalenthq.com/) | ||
VITE_COVALENT_API_KEY="" | ||
|
||
# Optional | ||
# Toggle to "true" to use Olympus Give features | ||
VITE_GIVE_ENABLED="true" | ||
# This should be toggled to "true" if you need to use the mock sOHM contract | ||
# (which allows for on-demand rebasing) | ||
VITE_MOCK_SOHM_ENABLED="false" | ||
|
||
# Optional | ||
# Google Analytics (https://analytics.google.com/) | ||
VITE_GOOGLE_ANALYTICS_API_KEY="" | ||
# Google Analytics 4 API Key | ||
VITE_GA_4_API_KEY="" | ||
|
||
# Optional | ||
# If you run your own node, you can provide connection url(s) to that node. | ||
|
@@ -25,14 +17,6 @@ VITE_FANTOM_TESTNET_NODE_URL="" | |
VITE_POLYGON_NODE_URL="" | ||
VITE_POLYGON_TESTNET_NODE_URL="" | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why do we need Polygon? |
||
# Olympus Give | ||
# These 2 settings should be toggled to "true" if you want to use Olympus Give features | ||
VITE_GIVE_ENABLED="true" | ||
VITE_GIVE_GRANTS_ENABLED="true" | ||
|
||
brightiron marked this conversation as resolved.
Show resolved
Hide resolved
|
||
# This should be toggled to "true" if you need to use the mock sOHM contract | ||
# (which allows for on-demand rebasing) | ||
VITE_MOCK_SOHM_ENABLED="false" | ||
|
||
VITE_ARBITRUM_NODE_URL="" | ||
VITE_ARBITRUM_TESTNET_NODE_URL="" | ||
|
@@ -41,4 +25,7 @@ VITE_AVALANCHE_NODE_URL="" | |
VITE_AVALANCHE_TESTNET_NODE_URL="" | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same question here. Why do we need this in code? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can probably remove these and only support mainnet and arbitrum on the network selector. |
||
# Get a wallet connect project id here: https://cloud.walletconnect.com | ||
VITE_WALLETCONNECT_PROJECT_ID="" | ||
VITE_WALLETCONNECT_PROJECT_ID="" | ||
|
||
#Subgraph URL for Protocol Metrics. If not set Protocol Metrics will not be displayed. | ||
VITE_WG_PUBLIC_NODE_URL="" |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,6 +21,7 @@ import StagingNotification from "src/components/StagingNotification"; | |
import { StakeVersionContainer } from "src/components/StakeVersionContainer"; | ||
import TopBar from "src/components/TopBar/TopBar"; | ||
import { shouldTriggerSafetyCheck } from "src/helpers"; | ||
import { Environment } from "src/helpers/environment/Environment/Environment"; | ||
import { useGoogleAnalytics } from "src/hooks/useGoogleAnalytics"; | ||
import useTheme from "src/hooks/useTheme"; | ||
import { chains } from "src/hooks/wagmi"; | ||
|
@@ -202,6 +203,8 @@ function App() { | |
if (isSidebarExpanded) handleSidebarClose(); | ||
}, [location]); | ||
|
||
const protocolMetricsEnabled = Boolean(Environment.getWundergraphNodeUrl()); | ||
const defaultRoute = protocolMetricsEnabled ? "/dashboard" : "/my-balances"; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we just make the default route always be There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes. just hadn't historically. so didn't wanna break functionality |
||
return ( | ||
<StyledDiv> | ||
<RainbowKitProvider | ||
|
@@ -231,7 +234,7 @@ function App() { | |
<Suspense fallback={<div></div>}> | ||
<Routes> | ||
<Route path="my-balances" element={<MyBalances />} /> | ||
<Route path="/" element={<Navigate to="/dashboard" />} /> | ||
<Route path="/" element={<Navigate to={defaultRoute} />} /> | ||
<Route | ||
path="/stake" | ||
element={<StakeVersionContainer setMigrationModalOpen={setMigrationModalOpen} />} | ||
|
This file was deleted.
This file was deleted.
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.
Why do we need both GA?