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

Cleanup Unused Deps #3039

Open
wants to merge 21 commits into
base: develop
Choose a base branch
from
Open

Cleanup Unused Deps #3039

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
e580ce9
remove react-step-progress-bar
brightiron Nov 26, 2023
242a0c3
remove react-uid
brightiron Nov 26, 2023
0a25ba8
remove react-markdown
brightiron Nov 26, 2023
a524808
remove class-validator
brightiron Nov 26, 2023
efd2fbb
remove unused graphql deps
brightiron Nov 26, 2023
6ea943c
remove unused deps
brightiron Nov 26, 2023
9579eec
more unused deps
brightiron Nov 26, 2023
d262b15
more unused deps and tests
brightiron Nov 26, 2023
87d9b52
wip
brightiron Nov 28, 2023
76cb989
Merge branch 'develop' into cleanupDeps
brightiron Dec 19, 2023
02087d5
remove unused packages
brightiron Dec 19, 2023
18568ed
add back coverage-v8
brightiron Dec 19, 2023
bf67535
revert
brightiron Dec 19, 2023
8eac788
Merge branch 'develop' into cleanupDeps
brightiron Dec 22, 2023
b176920
Merge branch 'develop' into cleanupDeps
brightiron Dec 22, 2023
e4fc216
formatting
brightiron Dec 22, 2023
c40b1ae
Merge branch 'develop' into cleanupDeps
brightiron Dec 22, 2023
07966ae
Merge branch 'develop' into cleanupDeps
brightiron Dec 22, 2023
3be44e5
updated lock
brightiron Dec 22, 2023
7153913
Merge branch 'develop' into cleanupDeps
brightiron Dec 22, 2023
d1c3a94
Merge branch 'develop' into cleanupDeps
brightiron Dec 22, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 0 additions & 29 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,29 +40,22 @@
"@reduxjs/toolkit": "^1.9.3",
"@tanstack/react-query": "^4.2.3",
"@types/recharts": "^1.8.29",
"@vitejs/plugin-react-swc": "^3.5.0",
"@vitest/coverage-v8": "^0.34.6",
"@wundergraph/react-query": "^0.9.27",
"axios": "^1.6.2",
"class-validator": "^0.14.0",
"date-fns": "^2.30.0",
"date-fns-tz": "^2.0.0",
"ethers": "^5.7.2",
"get-value": "^3.0.1",
"graphql": "^15.8.0",
"graphql-request": "^6.1.0",
"graphql-tag": "^2.0.0",
"luxon": "^3.4.4",
"react": "18.2.0",
"react-dom": "18.2.0",
"react-ga": "^3.3.1",
"react-ga4": "^2.1.0",
"react-hot-toast": "^2.4.1",
"react-markdown": "^8.0.7",
"react-redux": "^8.0.5",
"react-router-dom": "^6.21.1",
"react-step-progress-bar": "^1.0.3",
"react-uid": "^2.3.2",
"recharts": "^2.10.3",
"tinycolor2": "^1.6.0",
"typescript": "^5.3.3",
Expand All @@ -72,38 +65,27 @@
"@esbuild-plugins/node-globals-polyfill": "^0.2.3",
"@esbuild-plugins/node-modules-polyfill": "^0.2.2",
"@ethersproject/abi": "^5.7.0",
"@ethersproject/bytes": "^5.6.1",
"@redocly/cli": "^1.6.0",
"@tanstack/react-query-devtools": "^4.19.1",
"@testing-library/dom": "^9.3.3",
"@testing-library/react": "^14.1.2",
"@testing-library/user-event": "^14.5.1",
"@typechain/ethers-v5": "^11.1.2",
"@types/css-mediaquery": "^0.1.4",
"@types/get-value": "^3.0.5",
"@types/graphql": "^14.5.0",
"@types/luxon": "^3.3.7",
"@types/markdown-it": "^13.0.7",
"@types/node": "^20.10.5",
"@types/react": "^18.2.45",
"@types/react-dom": "^18.2.18",
"@types/react-router-dom": "^5.3.3",
"@types/shelljs": "^0.8.15",
"@types/tinycolor2": "^1.4.6",
"@types/uuid": "^9.0.7",
"@typescript-eslint/eslint-plugin": "^5.62.0",
"@typescript-eslint/parser": "^5.62.0",
"@vitejs/plugin-react": "^4.2.1",
"@vitest/ui": "^0.34.7",
"assert": "^2.1.0",
"autoprefixer": "^10.4.15",
"css-mediaquery": "^0.1.2",
"dotenv": "^16.3.1",
"eslint": "^8.56.0",
"eslint-config-airbnb": "^19.0.4",
"eslint-config-prettier": "^9.1.0",
"eslint-plugin-import": "^2.29.1",
"eslint-plugin-jsx-a11y": "^6.8.0",
"eslint-plugin-no-relative-import-paths": "^1.5.3",
"eslint-plugin-prettier": "^5.1.1",
"eslint-plugin-react": "^7.33.2",
Expand All @@ -112,27 +94,16 @@
"eslint-plugin-unused-imports": "^3.0.0",
"fast-check": "^3.15.0",
"happy-dom": "^12.10.3",
"https-browserify": "^1.0.0",
"husky": "^8.0.3",
"jsdom": "^21.1.1",
"less-plugin-npm-import": "^2.1.0",
"lint-staged": "^14.0.1",
"node-watch": "^0.7.4",
"orval": "^6.23.0",
"os-browserify": "^0.3.0",
"postcss": "^8.4.32",
"prettier": "^3.1.1",
"process": "^0.11.10",
"prop-types": "^15.8.1",
"resize-observer-polyfill": "^1.5.1",
"rollup-plugin-polyfill-node": "^0.13.0",
"sass": "^1.69.5",
"shelljs": "^0.8.5",
"stream-browserify": "^3.0.0",
"stream-http": "^3.2.0",
"type-graphql": "^1.1.1",
"typechain": "^8.3.2",
"url": "^0.11.1",
"vite": "^4.4.12",
"vite-plugin-svgr": "^4.2.0",
"vite-plugin-transform": "^2.0.1",
Expand Down
27 changes: 0 additions & 27 deletions src/types/react-step-progress-bar.d.ts
View file
Open in desktop

This file was deleted.

122 changes: 0 additions & 122 deletions tests/e2e/testHelpers.ts
View file
Open in desktop

This file was deleted.

9 changes: 0 additions & 9 deletions tests/e2e/tsconfig.json
View file
Open in desktop

This file was deleted.

13 changes: 6 additions & 7 deletions vite.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ import react from "@vitejs/plugin-react";
import polyfillNode from "rollup-plugin-polyfill-node";
import { defineConfig } from "vite";
import svgrPlugin from "vite-plugin-svgr";
import viteTsconfigPaths from "vite-tsconfig-paths";
import transformPlugin from "vite-plugin-transform";
import viteTsconfigPaths from "vite-tsconfig-paths";

export default ({ mode }) => {
return defineConfig({
Expand All @@ -20,12 +20,11 @@ export default ({ mode }) => {
tStart: "%{",
tEnd: "}%",
replace: {
"COOLER_LOANS_API_ENDPOINT":
process.env.VITE_COOLER_LOANS_API_ENDPOINT ?
process.env.VITE_COOLER_LOANS_API_ENDPOINT :
mode === "development" ?
"https://olympus-cooler-loans-api-dev.web.app" : // Avoids CORS errors during local development
"https://olympus-cooler-loans-api-prod.web.app", // Used in production builds
COOLER_LOANS_API_ENDPOINT: process.env.VITE_COOLER_LOANS_API_ENDPOINT
? process.env.VITE_COOLER_LOANS_API_ENDPOINT
: mode === "development"
? "https://olympus-cooler-loans-api-dev.web.app" // Avoids CORS errors during local development
: "https://olympus-cooler-loans-api-prod.web.app", // Used in production builds
Copy link
Contributor

Choose a reason for hiding this comment

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

What is hosting this? Why do we need a separate endpoint to serve Cooler Loans?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is for cooler loan metrics subgraph not for cooler loans. jem set this up.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@0xJem could you comment on this piece.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is hosted on Google Cloud

Repo is here: https://github.com/OlympusDAO/cooler-loans-api

Calculating the timeseries data in the frontend made it very slow, so it's shoved into an API that updates the data every 5 minutes

},
}),
],
Expand Down
Loading
Loading