Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/sc 197 statistics #21

Merged
merged 6 commits into from
Jan 7, 2024
Merged

Feature/sc 197 statistics #21

merged 6 commits into from
Jan 7, 2024

Conversation

lgaljer20
Copy link
Collaborator

Created functionality for admin to get a list of events for specified charger based on provided month and year. List will be filled with events and basic data about each event (startTime, endTime and charged volume).
Changes made:

  1. Added method GetStatistics in event interface
  2. Implemented GetStatistics in event service
  3. Implemented endpoint in event controller
  4. Created unit tests for newly added method for controller and service

All previous tests and newly added tests have successfully passed. ✅

@lgaljer20 lgaljer20 added the enhancement New feature or request label Jan 7, 2024
Tests failed in CI/CD pipeline because of the name of the month
Copy link
Collaborator

@lraknic20 lraknic20 left a comment

Choose a reason for hiding this comment

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

Everything looks good!

Copy link
Collaborator

@irosic20 irosic20 left a comment

Choose a reason for hiding this comment

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

Everything looks good. Keep up the good work!

Copy link
Owner

@mkajic20 mkajic20 left a comment

Choose a reason for hiding this comment

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

Everything seems good.

@mkajic20 mkajic20 merged commit f918555 into develop Jan 7, 2024
1 check passed
@lgaljer20 lgaljer20 self-assigned this Jan 10, 2024
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.

4 participants