Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Voeg DAO interfaces + implementaties toe #21

Merged
merged 31 commits into from
Feb 27, 2024
Merged

Voeg DAO interfaces + implementaties toe #21

merged 31 commits into from
Feb 27, 2024

Conversation

ALBERICLOOS
Copy link
Contributor

#7

@lbarraga lbarraga self-requested a review February 25, 2024 15:09
lbarraga
lbarraga suggested changes Feb 25, 2024
View reviewed changes
backend/db/implementation/SqlAdminDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlAdminDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlGroupDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlGroupDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlGroupDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlGroupDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlSubjectDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlTeacherDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlUserDAO.py Outdated Show resolved Hide resolved
backend/fill_database_mock.py Outdated Show resolved Hide resolved
@lbarraga lbarraga mentioned this pull request Feb 25, 2024
Merged
Base automatically changed from add-models to backend_file_structure_POC February 25, 2024 20:38
Base automatically changed from backend_file_structure_POC to main February 25, 2024 21:07
@ALBERICLOOS ALBERICLOOS changed the title Voeg DAO interfaces toe Voeg DAO interfaces + implementaties toe Feb 26, 2024
@lbarraga lbarraga self-requested a review February 27, 2024 06:30
@lbarraga lbarraga added this to the Milestone 1 milestone Feb 27, 2024
lbarraga
lbarraga suggested changes Feb 27, 2024
View reviewed changes
backend/db/implementation/SqlAdminDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlGroupDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlProjectDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlProjectDAO.py Show resolved Hide resolved
backend/db/implementation/SqlGroupDAO.py Show resolved Hide resolved
backend/db/implementation/SqlUserDAO.py Outdated Show resolved Hide resolved
@ALBERICLOOS ALBERICLOOS requested a review from lbarraga February 27, 2024 12:03
lbarraga
lbarraga previously approved these changes Feb 27, 2024
View reviewed changes
Copy link
Contributor

@lbarraga lbarraga left a comment

Choose a reason for hiding this comment

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

Goed gedaan

@msathieu msathieu self-requested a review February 27, 2024 13:07
msathieu
msathieu suggested changes Feb 27, 2024
View reviewed changes
backend/db/implementation/SqlGroupDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlSubjectDAO.py Outdated Show resolved Hide resolved
backend/db/implementation/SqlSubjectDAO.py Outdated Show resolved Hide resolved
@ALBERICLOOS ALBERICLOOS requested a review from msathieu February 27, 2024 14:05
@ALBERICLOOS ALBERICLOOS merged commit fe3230d into main Feb 27, 2024
1 check passed
@ALBERICLOOS ALBERICLOOS deleted the dao branch February 27, 2024 14:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@msathieu msathieu msathieu approved these changes

@lbarraga lbarraga lbarraga left review comments

Assignees

@ALBERICLOOS ALBERICLOOS

Labels
None yet
Projects
None yet
Milestone
Milestone 1
Development

Successfully merging this pull request may close these issues.
3 participants
@ALBERICLOOS @msathieu @lbarraga