#724-restructuring-of-roombooking-service-code #882

Merged
a24julot merged 16 commits from #724-restructuring-of-roombooking-service-code into team_1_week_5 2026-05-12 07:56:30 +00:00
Collaborator

Restructured the whole of roombooking to use the new determined structure

!NOTICE!
You use the old url's to reach the endpoints

PR for #724

Restructured the whole of roombooking to use the new determined structure !NOTICE! You use the old url's to reach the endpoints PR for #724
Merely renaming the folder doesn't update the files that are generated when creating a new dotnet project, so recreating it by using a template was necessary.
roombooking keeps restarting because of som sort of dll error, fixed it by changing the name to camelcase
have created all the neccesary directories and files and have started filling the files with code, have finished the majority of database logic
have started on services
Did dokumentation on database logic
Added the rest of the functionality to the service
Changed so that the booking class uses upercase for first letter
Fixed some syntax errors
Moved the dokumentation of database queries to its interface
added code for endpoints
added code for program.cs
the service builds and the endpoints sems to work
Fixed the error where there are extra bookings created with same values of existing, where i want to create 8 there end upp 15 total
Removed the old backup code
a24sakma requested changes 2026-05-11 12:30:00 +00:00
Dismissed
a24sakma left a comment
Collaborator

What works:

  • The structure follows the model
  • Endpoints
  • Interfaces
  • database logic

What needs to be done

  1. update readme
  2. check the comments where I added

When running the application after a clean start I get an error image
but when I click on another card then this card then it works. It is probably a frontend issue but I needed to raise

## What works: - The structure follows the model - Endpoints - Interfaces - database logic ## What needs to be done 1. update readme 2. check the comments where I added When running the application after a clean start I get an error ![image](/attachments/3175d19d-e023-4f5f-97a7-4d37edba30aa) but when I click on another card then this card then it works. It is probably a frontend issue but I needed to raise
@ -15708,23 +15708,6 @@
}
}
},
"node_modules/tailwindcss/node_modules/yaml": {
Collaborator

Was this relevant to remove?

Was this relevant to remove?
Author
Collaborator

dont even know where that got from, have not touched frontend,
was probably added/removed when i did a merge and run the rebuild script

dont even know where that got from, have not touched frontend, was probably added/removed when i did a merge and run the rebuild script
Collaborator

I think it is fine because the libraries installs through the package.json and package-lock-json keeps only hash with defined versions from package.json.
It will be reinstalled when rebuilding the project.

I think it is fine because the libraries installs through the package.json and package-lock-json keeps only hash with defined versions from package.json. It will be reinstalled when rebuilding the project.
@ -0,0 +16,4 @@
var dataSource = NpgsqlDataSource.Create(ConnectionString);
builder.Services.AddSingleton(dataSource);
// builder.Services.AddMemoryCache();
Collaborator

this commented line can be removed?

this commented line can be removed?
a24antej marked this conversation as resolved
a24sakma approved these changes 2026-05-11 13:11:45 +00:00
Author
Collaborator

This PR should be ready to be merged into the team branch

This PR should be ready to be merged into the team branch
a24julot merged commit 54b931fe19 into team_1_week_5 2026-05-12 07:56:30 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!882
No description provided.