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

Feat/db #66

Open
wants to merge 6,072 commits into
base: master
Choose a base branch
from
Open

Feat/db #66

wants to merge 6,072 commits into from

Conversation

Bad3r
Copy link
Owner

@Bad3r Bad3r commented Jun 16, 2023

No description provided.

logseq-cldwalker and others added 30 commits January 29, 2025 17:57
Add test to confirm multiple imports are idempotent.
Also fix sqlite.build not handling valid value for :db/cardinality
from properties. Also fix user specified :pages-and-blocks not building
with :build-existing-tx?
Gracefully handle invalid EDN and property conflicts
between existing and new blocks correctly. Imports
into a block worked fine because it always existed but
imports into new blocks were failing
Works across graphs for basic top-level blocks. Generalized import
command to work with block+page and other future exports
or graph. Also add tests for fresh page import
root needed to be re-rendered like with file graph import
for both existing or other graphs. Introduced an enhanced `[:build/page]` ref for
this
refs from sqlite.build as they have been replaced with
an improved [:build/page ...]
Also enhance :build/page refs so their tags export
property values. Import works well for new graphs
Affected this status or any user ones with spaces in name.
Fixes logseq/db-test#215
e.g. Status. Also tweak class-parent to not needlessly emit Root
which is the default
Needed for export+import of refs
All ref types handled except class ref. Also fixed most block types not
persisting uuid when :build-existing-tx? set
by creating test helpers. Also add 2nd import assertion on two more tests
now that's easy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants