Skip to content

refactor(dataset-store): wip#753

Closed
LNSD wants to merge 1 commit intomainfrom
lnsd/feat-dataset-store-manifest-store
Closed

refactor(dataset-store): wip#753
LNSD wants to merge 1 commit intomainfrom
lnsd/feat-dataset-store-manifest-store

Conversation

@LNSD
Copy link
Contributor

@LNSD LNSD commented Sep 17, 2025

Separate manifest-specific concerns from the main dataset-store module to improve code organization and maintainability.

  • Create new manifests.rs module with dedicated CRUD operations
  • Extract manifest tests to separate test modules for better organization
  • Refactor main dataset-store module to use new manifest abstractions
  • Update admin API registration handler to use new manifest interface
  • Remove redundant manifest derivation logic from common crate

@LNSD LNSD self-assigned this Sep 17, 2025
@LNSD LNSD force-pushed the lnsd/feat-dataset-store-manifest-store branch 9 times, most recently from bd8e31a to 97ef3b3 Compare September 18, 2025 09:59
Separate manifest-specific concerns from the main dataset-store module to improve code organization and maintainability.

- Create new `manifests.rs` module with dedicated CRUD operations
- Extract manifest tests to separate test modules for better organization
- Refactor main dataset-store module to use new manifest abstractions
- Update admin API registration handler to use new manifest interface
- Remove redundant manifest derivation logic from common crate

Signed-off-by: Lorenzo Delgado <lorenzo@edgeandnode.com>
@LNSD LNSD force-pushed the lnsd/feat-dataset-store-manifest-store branch from 97ef3b3 to 1f658e9 Compare September 18, 2025 10:01
@LNSD
Copy link
Contributor Author

LNSD commented Sep 24, 2025

Closing this PR in favour of #796

@LNSD LNSD closed this Sep 24, 2025
@LNSD LNSD deleted the lnsd/feat-dataset-store-manifest-store branch October 14, 2025 15:40
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.

1 participant