Skip to content

Commit

Permalink
fix: externalize deps (#392)
Browse files Browse the repository at this point in the history
* fix: externalize deps

* docs: add changeset
  • Loading branch information
NWRichmond committed Jun 9, 2023
1 parent aa293e4 commit 254f5b0
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 6 deletions.
5 changes: 5 additions & 0 deletions .changeset/mighty-balloons-cry.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@nacelle/storefront-sdk': patch
---

Fixes an internal issue with `@nacelle/storefront-sdk`'s build process, which was causing some external package code to be inadvertently bundled in the `@nacelle/storefront-sdk` package code. This issue both increased the package size and caused issues in Nuxt 2 projects.
10 changes: 4 additions & 6 deletions packages/storefront-sdk/vite.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,16 +17,14 @@ export const config: UserConfig = {
rollupOptions: {
external: [
'@urql/core',
'@urql/exchange-persisted-fetch',
'@urql/exchange-retry',
'graphql'
'@urql/exchange-persisted',
'@urql/exchange-retry'
],
output: {
globals: {
'@urql/core': 'Urql',
'@urql/exchange-persisted-fetch': 'UrqlExchangePersistedFetch',
'@urql/exchange-retry': 'UrqlExchangeRetry',
graphql: 'Graphql'
'@urql/exchange-persisted': 'UrqlExchangePersisted',
'@urql/exchange-retry': 'UrqlExchangeRetry'
}
}
}
Expand Down

4 comments on commit 254f5b0

@vercel
Copy link

@vercel vercel bot commented on 254f5b0 Jun 9, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

nacelle-gatsby-reference-store – ./reference-stores/gatsby

nacelle-gatsby-reference-store.vercel.app
nacelle-gatsby-reference-store-git-main-nacelle.vercel.app
nacelle-gatsby-reference-store-nacelle.vercel.app

@vercel
Copy link

@vercel vercel bot commented on 254f5b0 Jun 9, 2023

Choose a reason for hiding this comment

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

@vercel
Copy link

@vercel vercel bot commented on 254f5b0 Jun 9, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

nacelle-next-reference-store – ./reference-stores/next

nacelle-next-reference-store-git-main-nacelle.vercel.app
nacelle-next-reference-store.vercel.app
nacelle-next-reference-store-nacelle.vercel.app

@vercel
Copy link

@vercel vercel bot commented on 254f5b0 Jun 9, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

nacelle-gatsby-starter – ./starters/gatsby

nacelle-gatsby-starter-nacelle.vercel.app
nacelle-gatsby-starter.vercel.app
nacelle-gatsby-starter-git-main-nacelle.vercel.app

Please sign in to comment.