Skip to content

Conversation

@rawe0
Copy link
Member

@rawe0 rawe0 commented Oct 22, 2025

This PR includes:

  • Conversion of the SQL tables to a GORM model, IDs are unique across public links, normal shares, and OCM shares.
    • Some refactoring of the OCM tables (protocols and access methods)
  • Corresponding sql driver for access has been implemented using GORM
  • Tests with basic coverage have been implemented.

Note: before merging a puppet change is needed (commit is ready) and the ocm share tables might need to be recreated.

@update-docs
Copy link

update-docs bot commented Oct 22, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@rawe0 rawe0 force-pushed the feat/gormify-ocm-sql-tables branch 4 times, most recently from e00ad89 to ef96e80 Compare October 28, 2025 16:26
@rawe0 rawe0 force-pushed the feat/gormify-ocm-sql-tables branch 22 times, most recently from 8330f51 to 9170f5c Compare November 6, 2025 16:09
@rawe0 rawe0 marked this pull request as ready for review November 6, 2025 16:10
@rawe0 rawe0 requested a review from jessegeens as a code owner November 6, 2025 16:10
@rawe0 rawe0 force-pushed the feat/gormify-ocm-sql-tables branch 15 times, most recently from 3ada0e7 to 533d75d Compare November 7, 2025 16:19
@rawe0 rawe0 force-pushed the feat/gormify-ocm-sql-tables branch from 533d75d to bcb2116 Compare November 7, 2025 16:20
@glpatcern glpatcern merged commit 47144b1 into master Nov 10, 2025
14 checks passed
@jessegeens jessegeens deleted the feat/gormify-ocm-sql-tables branch December 11, 2025 13:07
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.

3 participants