feat(cat-voices): adjust database to signed docs 0.0.4 #3791
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
damian-molinski
added this to the
M5: Voting & Delegation - Production Readiness milestone
damian-molinski
requested review from
LynxLynxx,
bstolinski and
dt-iohk
as code owners
…llaborators_adjustments_3769_part2
….com:input-output-hk/catalyst-voices into feat/db_collaborators_adjustments_3769_part2
…llaborators_adjustments_3769_part2
dt-iohk
reviewed
Dec 4, 2025
...ernal/catalyst_voices_repositories/test/src/database/migration/catalyst_database/native.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...ernal/catalyst_voices_repositories/test/src/database/migration/catalyst_database/native.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...ernal/catalyst_voices_repositories/test/src/database/migration/catalyst_database/native.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...catalyst_voices_repositories/test/src/database/dao/documents_v2_local_metadata_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...catalyst_voices_repositories/test/src/database/dao/documents_v2_local_metadata_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...catalyst_voices_repositories/test/src/database/dao/documents_v2_local_metadata_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...catalyst_voices_repositories/test/src/database/dao/documents_v2_local_metadata_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...catalyst_voices_repositories/test/src/database/dao/documents_v2_local_metadata_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
reviewed
Dec 4, 2025
...kages/internal/catalyst_voices_repositories/test/src/database/dao/documents_v2_dao_test.dart
Outdated
Show resolved
Hide resolved
dt-iohk
approved these changes
Dec 4, 2025
Contributor
dt-iohk
left a comment
dt-iohk
left a comment
There was a problem hiding this comment.
LGTM. There were other comments for minor stuff, consider it approved once you address them
LynxLynxx
approved these changes
Dec 4, 2025
damian-molinski
merged commit 26f1448
into
feat/db_collaborators_adjustments_3769
40 checks passed
damian-molinski
deleted the
feat/db_collaborators_adjustments_3769_part2
branch
damian-molinski
added a commit
that referenced
this pull request
Dec 5, 2025
* chore: port over parameters changes * feat(cat-voices): adjust database to signed docs 0.0.4 (#3791) * tables * wip * proposals dao tests * bump db schema * db index name fix * migration -> use createAll() * update migration * update migration * chore: remove print * update ref dto flatten implementation * migration toFlatten() * fix: proposals dao * fix: documents dao and local drafts dao * chore: clean up TODO * fix: failing tests * chore: markdown order * web: skips for db tests * fix: db tests on web * fix: analyzer * unskip tests * rename * CatalystId.toUri -> CatalystId.toString * use same instead of identical * use hasLength * metadata dao tests * use hasLength in documents v2 dao * feat: use hasLength in proposals v2 dao tests * fix(cat-voices): storing catIds with coma usernames (#3813) * tables * wip * proposals dao tests * bump db schema * db index name fix * migration -> use createAll() * update migration * update migration * chore: remove print * update ref dto flatten implementation * migration toFlatten() * fix: proposals dao * fix: documents dao and local drafts dao * chore: clean up TODO * fix: failing tests * chore: markdown order * web: skips for db tests * fix: db tests on web * fix: analyzer * unskip tests * chore: use converters * update tests * hello world test * fix: documents dao hasLength * feat(cat-voices): filtering proposals by original author (#3810) * tables * wip * proposals dao tests * bump db schema * db index name fix * migration -> use createAll() * update migration * update migration * chore: remove print * update ref dto flatten implementation * migration toFlatten() * fix: proposals dao * fix: documents dao and local drafts dao * chore: clean up TODO * fix: failing tests * chore: markdown order * web: skips for db tests * fix: db tests on web * fix: analyzer * unskip tests * feat: filtering author on first version * change filter name from author to originalAuthor * Adds originalAuthor filter to documentsDao * authorId -> originalAuthorId * originalAuthors * fix: spelling * use authors from documents for originalAuthors * feat(cat-voices): validating proposal submission action author (#3814) * tables * wip * proposals dao tests * bump db schema * db index name fix * migration -> use createAll() * update migration * update migration * chore: remove print * update ref dto flatten implementation * migration toFlatten() * fix: proposals dao * fix: documents dao and local drafts dao * chore: clean up TODO * fix: failing tests * chore: markdown order * web: skips for db tests * fix: db tests on web * fix: analyzer * unskip tests * feat: filtering author on first version * change filter name from author to originalAuthor * Adds originalAuthor filter to documentsDao * authorId -> originalAuthorId * originalAuthors * fix: spelling * docs * use authors from documents for originalAuthors * fix: campaign total ask tests * fix: proposals dao tests * test: proposals dao authors * fix: spelling * chore: extract _getValidActionsCTE()
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR updates, and finishes, db migration to signed docs 0.0.4 format (parameters and collaborators)
Related Issue(s)
Resolves #3769
Description of Changes
DocumentCollaboratorsandDocumentParameterstables for fast join lookupsDocumentsV2Dao,LocalDraftDocumentsV2DaoandProposalsV2Daoto work with parametersDocumentRefDtonow have additional format "Flatten". It's used to encode/decodeparametersfromdocuments_v2andlocal_documents_draftscolumns where it's also stored for easier readingRelated Pull Requests
Please confirm the following checks