Skip to content

Commit

Permalink
fix(admin-sdk): Remove polyfills and unused dependencies (#7434)
Browse files Browse the repository at this point in the history
  • Loading branch information
kasperkristensen committed May 23, 2024
1 parent 135772b commit 253bec6
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 606 deletions.
6 changes: 1 addition & 5 deletions packages/admin-next/admin-sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,14 +26,10 @@
"@medusajs/dashboard": "0.0.1",
"@vitejs/plugin-react": "^4.2.1",
"autoprefixer": "^10.4.16",
"commander": "^11.1.0",
"compression": "^1.7.4",
"deepmerge": "^4.3.1",
"glob": "^7.1.6",
"postcss": "^8.4.32",
"tailwindcss": "^3.3.6",
"vite": "^5.2.11",
"vite-plugin-node-polyfills": "^0.21.0"
"vite": "^5.2.11"
},
"peerDependencies": {
"express": "^4.18.2",
Expand Down
10 changes: 0 additions & 10 deletions packages/admin-next/admin-sdk/src/lib/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import { VIRTUAL_MODULES } from "@medusajs/admin-shared"
import path from "path"
import { Config } from "tailwindcss"
import type { InlineConfig } from "vite"
import { nodePolyfills } from "vite-plugin-node-polyfills"

import { BundlerOptions } from "../types"

Expand Down Expand Up @@ -58,15 +57,6 @@ export async function getViteConfig(
medusa({
sources: options.sources,
}),
/**
* TODO: Remove polyfills, they are currently only required for the
* `axios` dependency in the dashboard. Once we have the new SDK,
* we should remove this, and leave it up to the user to include
* polyfills if they need them.
*/
nodePolyfills({
include: ["crypto", "util", "stream"],
}),
],
}
}
Expand Down
Loading

0 comments on commit 253bec6

Please sign in to comment.