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 Conditional exports support #235

Closed
wants to merge 1 commit into from
Closed

Conversation

Gaubee
Copy link

@Gaubee Gaubee commented Sep 7, 2021

see doc

@jakearchibald
Copy link
Owner

I don't think this is enough, as the file extensions will need to change one way or the other. See jakearchibald/idb-keyval#126

@jakearchibald
Copy link
Owner

I'm going to try and fix this soon, using whatever solution I come to for idb-keyval

@jakearchibald
Copy link
Owner

I did this for idb-keyval in jakearchibald/idb-keyval#129. I'm going to leave it for a few days and see if it causes problems for anyone. If not, I'll do the same for this library.

@Feiyang1
Copy link

@jakearchibald any update? Love to have the exports support (especially the import condition) to make SSR easier using esm files.

@jakearchibald
Copy link
Owner

It's still on my radar. I'm happy with how it went with idb-keyval, so just waiting for the free time.

@jakearchibald
Copy link
Owner

Closing in favour of #242. It's also published as https://www.npmjs.com/package/idb/v/7.0.0-beta. I'd really appreciate some testing and feedback!

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.

3 participants