Skip to content

63 assignment repository - #174

Merged
brentjan merged 2 commits into
developmentfrom
63-assignment-repository
Mar 9, 2025
Merged

63 assignment repository#174
brentjan merged 2 commits into
developmentfrom
63-assignment-repository

Conversation

@brentjan

@brentjan brentjan commented Mar 9, 2025

Copy link
Copy Markdown
Contributor

Implemented the assignment repository and it's datasource, also added tests. This closes #63

@brentjan brentjan added the enhancement New feature or request label Mar 9, 2025
@brentjan brentjan added this to the Milestone 1 milestone Mar 9, 2025
@brentjan brentjan linked an issue Mar 9, 2025 that may be closed by this pull request
1 task
@brentjan
brentjan requested a review from bramcomyn March 9, 2025 12:22
@rvdkeere rvdkeere linked an issue Mar 9, 2025 that may be closed by this pull request
1 task

@bramcomyn bramcomyn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! Nice to see you took all previous comments and made sure they were adhered to in this PR! Great work there :)

@brieromb brieromb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done
I would like it if you would also accept mine :)

@brentjan
brentjan merged commit 6e7744f into development Mar 9, 2025
@thomasdejaeghere
thomasdejaeghere deleted the 63-assignment-repository branch March 11, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Backend] - Core - Repository interface Assignment Implementation [Backend] - Infrastructure - Repository Assignment Implementation

4 participants