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

Dexie Syncable with Dexie 3.0.0 #1048

Closed
oogghh opened this issue May 14, 2020 · 8 comments · Fixed by #1117
Closed

Dexie Syncable with Dexie 3.0.0 #1048

oogghh opened this issue May 14, 2020 · 8 comments · Fixed by #1117

Comments

@oogghh
Copy link

oogghh commented May 14, 2020

Am i correct to assume the current version of Dexie syncable will not work with Dexie 3.0.0?
I get a lot of namespace errors, and import errors such as
cannot augment module 'Dexie' with value exports because it resolves to a non-module entity.

@dfahlander
Copy link
Collaborator

Hi! They are runtime compatible but there might be typing issues.

@oogghh
Copy link
Author

oogghh commented May 14, 2020

Okay. Thank you very much

@oogghh oogghh closed this as completed May 14, 2020
@oogghh
Copy link
Author

oogghh commented May 25, 2020

Hello, Sorry for re-opening this one. Do you have any suggestions for fixing the typing issues for this? Although the package is working, we're unable to run any build pipelines due to the typing issues.

Thank you

@dfahlander
Copy link
Collaborator

Any contribution of the typings are welcome. If someone could provide me with a PR with the changes needed in Dexie.Syncable.d.ts and api.d.ts, I can release a new version of the addon.

@dfahlander
Copy link
Collaborator

I'm working on this now

@dfahlander dfahlander reopened this Sep 4, 2020
@oogghh
Copy link
Author

oogghh commented Sep 4, 2020

Awesome. Thank you very much

@dfahlander
Copy link
Collaborator

dfahlander commented Sep 8, 2020

I've published new versions of dexie-observable and dexie-syncable on npm that should be type-wise compatible with Dexie 3.0. Any feedback on these would be much apprechiated:

npm i dexie-observable@next dexie-syncable@next

Should give you versions 3.0.0-beta.9 on both of them.

@dfahlander
Copy link
Collaborator

Anyone who have tested the addons?

npm i dexie-observable@next dexie-syncable@next

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 a pull request may close this issue.

2 participants