search

nextapps-de / flexsearch

Next-Generation full text search library for Browser and Node.js
Apache License 2.0
12.54k stars 492 forks source link

Update docs for usage with ESM / ES6 #398

Closed danawoodman closed 10 months ago

danawoodman commented 1 year ago

The existing docs show a relative path, not the actual npm module bundle path.

I believe this should be correct for ESM/ES6 type environments and will hopefully prevent confusion as found in #341 etc.

jorgebg commented 1 year ago

This worked for me, thanks a million! I think updating the docs would help until https://github.com/nextapps-de/flexsearch/pull/343 is finished.

danawoodman commented 1 year ago

@jorgebg glad it helped! Hopefully @ts-thomas has a chance to review and merge (or improve) so other devs aren't confused.

sroertgen commented 1 year ago

@danawoodman You're awesome. Thank you for your effort

robertheessels commented 1 year ago

Awesome pull request. Please merge!

This is the way to import Flexsearch in a Vite (etc.) project.

danawoodman commented 1 year ago

Glad it helped yall!

danawoodman commented 1 year ago

Following up on this, any chance it gets merged in so it can help others?

ts-thomas commented 10 months ago

Thanks a lot!