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

Update docs for usage with ESM / ES6 #398

Merged
merged 2 commits into from
Jan 6, 2024

Conversation

danawoodman
Copy link
Contributor

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.

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 nextapps-de#341 etc.
@jorgebg
Copy link

jorgebg commented Aug 10, 2023

This worked for me, thanks a million! I think updating the docs would help until #343 is finished.

@danawoodman
Copy link
Contributor Author

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

@sroertgen
Copy link

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

@robertheessels
Copy link

Awesome pull request. Please merge!

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

@danawoodman
Copy link
Contributor Author

Glad it helped yall!

@danawoodman
Copy link
Contributor Author

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

@ts-thomas ts-thomas merged commit 0921303 into nextapps-de:master Jan 6, 2024
@ts-thomas
Copy link
Contributor

Thanks a lot!

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.

5 participants