#899-Implement-logger-for-energyService #913

Merged
a24julot merged 8 commits from #899-Implement-logger-for-energyService into team_1_week_5 2026-05-12 13:03:53 +00:00
Collaborator

Implement logger in energyService

Environment

This was developed on an Arch-based Linux distribution.

What has been done?

  • service/energyService/energyService/Program.cs
    • I cleanded up the code a bit by removing duplicates of some lines and renamed some varibles to be convention complient.
  • service/energyService/energyService/Endpoints/EnergyEndpoints.cs
    • Updated the file structure to be more testable.
    • Added logging using Ilogger
  • service/energyService/energyService/Services/EnergyService.cs
    • Updated argument name for the class.
    • Added logging.
    • Added a help method to logg and return the values to do DRY (don't repeat yourself), since does lines where in each method.
  • service/energyService/energyService/DatabaseLogic/DatabaseQueries.cs
    • Added logging by removing console.log and replacing them with a logger.
  • General:
    • Formated, and used explicit types instead of var.

Issue:

General: #899

# Implement logger in energyService ## Environment This was developed on an Arch-based Linux distribution. ## What has been done? * **service/energyService/energyService/Program.cs** * I cleanded up the code a bit by removing duplicates of some lines and renamed some varibles to be convention complient. * **service/energyService/energyService/Endpoints/EnergyEndpoints.cs** * Updated the file structure to be more testable. * Added logging using Ilogger * **service/energyService/energyService/Services/EnergyService.cs** * Updated argument name for the class. * Added logging. * Added a help method to logg and return the values to do DRY (don't repeat yourself), since does lines where in each method. * **service/energyService/energyService/DatabaseLogic/DatabaseQueries.cs** * Added logging by removing console.log and replacing them with a logger. * General: * Formated, and used explicit types instead of var. ## Issue: General: #899
a24julot 2026-05-12 10:17:46 +00:00
a24sakma approved these changes 2026-05-12 11:55:26 +00:00
a24sakma left a comment
Collaborator

Great job!

  • Great clean up!
Great job! - Great clean up!
a24julot merged commit 15e3498f62 into team_1_week_5 2026-05-12 13:03:53 +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!913
No description provided.