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.

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

fix: v2 export issue #107

Merged
merged 4 commits into from
Nov 11, 2024
Merged

fix: v2 export issue #107

merged 4 commits into from
Nov 11, 2024

Conversation

martinalbert
Copy link
Contributor

@martinalbert martinalbert commented Oct 31, 2024

This PR resolves #97 and resolves #105.

It split builds into UMD and ES module builds which ensures import Butter from "buttercms" works.

It also adds development builds and leverages Tenser plugin for webpack instead of uglify.

Along this, it updates dependencies, as shown in table bellow:

Overview of version changes:

Dependency Old version New version
webpack 5.89.0 5.95.0
msw 2.0.11 2.6.0
uglify 3.17.4 none
terser-webpack-plugin none 5.3.10

@ViolanteCodes
Copy link
Contributor

@courcelan can you take a look at this when you get a chance? Thank you

Copy link
Contributor

@marcinpece-apptension marcinpece-apptension left a comment

Choose a reason for hiding this comment

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

Issue with importing solved. 🎉
I approved, but let @courcelan or @ViolanteCodes look at it too
https://share.zight.com/RBuRD4KB

@ViolanteCodes
Copy link
Contributor

@marcinpece-apptension i don't see anywhere else we are using this feature, unless i am missing something. Are you seeing anywhere? I imagined we moved over to either css or bootstrap for smooth scrolling at some point when it was necessary? WDYT?

@marcinpece-apptension
Copy link
Contributor

I guess you ask here about other PR

@ViolanteCodes
Copy link
Contributor

I guess you ask here about other PR

Yes 🤦

Copy link
Contributor

@courcelan courcelan left a comment

Choose a reason for hiding this comment

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

This looks perfect and works well. Like the split out for dev and production.

LGTM

@ViolanteCodes ViolanteCodes merged commit cd899b5 into master Nov 11, 2024
@ViolanteCodes ViolanteCodes deleted the fix/v2-export-issue branch November 11, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can no longer import Butter in a NextJS project Please add V2 API typings to package
4 participants