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: vite buffer polyfill in frontend-only example #136

Closed
wants to merge 2 commits into from

Conversation

moaazsidat
Copy link
Contributor

Motivation

frontend-only example has a bug where it's missing the Buffer polyfill resulting in the following error

Screenshot 2024-02-14 at 12 35 56 PM

Change Summary

Adds vite-plugin-node-polyfills package to frontend-only example and updates vite config to use the buffer polyfill

Merge Checklist

Choose all relevant options below by adding an x now or at any time before submitting for review

  • PR title adheres to the conventional commits standard
  • PR has a changeset
  • PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore)
  • PR includes documentation if necessary
  • All commits have been signed

Additional Context

If this is a relatively large or complex change, provide more details here that will help reviewers

@moaazsidat moaazsidat changed the title fix: vite buffer polyfill fix: vite buffer polyfill in frontend-only example Feb 14, 2024
@horsefacts
Copy link
Collaborator

Thank you!

@horsefacts horsefacts mentioned this pull request Feb 14, 2024
5 tasks
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.

2 participants