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

fix levelgraph #73

Open
scenaristeur opened this issue Jun 26, 2023 · 2 comments
Open

fix levelgraph #73

scenaristeur opened this issue Jun 26, 2023 · 2 comments
Labels
more information needed Further information is requested question Support or open question(s)

Comments

@scenaristeur
Copy link

scenaristeur commented Jun 26, 2023

i need help to fix levelgraph, see
levelgraph/levelgraph#197
according to https://github.com/Level/level/blob/master/UPGRADING.md#streams-have-moved
i've tried to replace leveldb.createValueStream() with leveldb.values() but it does not work

i've build a test for it https://github.com/scenaristeur/levelgraph/blob/master/test/persist-level/persist_properties_spec%20copy.js

thxs

@vweevers vweevers added question Support or open question(s) more information needed Further information is requested labels Jun 26, 2023
@scenaristeur
Copy link
Author

i was using levelgraph that use level for one of my project, but it seems that since level upgrade, levelgraph does not work anymore because it uses the old createValueStream() function, I would like to update the code of levelgrph, but i don't know what/where i have to change
https://github.com/levelgraph/levelgraph/blob/cae97978e4ea2c2db5ab2ef891ce9945325beec5/lib/levelgraph.js#L59 to fix the issue

@vweevers
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more information needed Further information is requested question Support or open question(s)
Projects
None yet
Development

No branches or pull requests

2 participants