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

Add 'events' package dependency #451

Merged
merged 3 commits into from
Mar 4, 2022
Merged

Add 'events' package dependency #451

merged 3 commits into from
Mar 4, 2022

Conversation

mbullington
Copy link
Contributor

@mbullington mbullington commented Feb 1, 2022

Fixes #441

New packagers like Vite don't automatically replace Node packages (events) with their browser equivalents, so this PR just installs the events NPM package so our code always has a fallback.

  • briefly describe the changes in this PR
  • write tests for all new functionality
  • run npm run docs and commit changes to API.md
  • update CHANGELOG.md with changes under master heading before merging

@mpothier mpothier merged commit a918b45 into main Mar 4, 2022
@mpothier mpothier deleted the events-dep branch March 4, 2022 14:57
janosh added a commit to sbsev/site that referenced this pull request Mar 5, 2022
* bump GH actions/setup-node v2 -> v3

* yarn remove cross-fetch + events (later now incl. by mapbox/mapbox-gl-geocoder#451)

* includen hidden links to all contenful pages in nav
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.

"Uncaught TypeError: EventEmitter is not a constructor" using vite
3 participants