Skip to content

Conversation

@Calebgisa72
Copy link
Contributor

@Calebgisa72 Calebgisa72 commented Nov 23, 2024

PR Description

This PR works on the logic of counting the attendance of a trainee and the number of open tickets of a trainee then supply it to the frontend.

Description of tasks that were expected to be completed

  • Show trainee attendance
  • Show trainee open tickets

How has this been tested?

  • Clone this repository and navigate to the project directory
  • Ensure that you have set your .env file using .env.example.

git clone https://github.com/atlp-rwanda/atlp-pulse-bn.git cd atlp-pulse-bn git checkout trainee-stats-ttl npm i npm run dev

@Calebgisa72 Calebgisa72 added Needs Reviews A pull request need reviews to be merged steppers Steppers Team labels Nov 23, 2024
@ceelogre ceelogre temporarily deployed to trainee-stats-ttl - devpulse-backend PR #439 November 23, 2024 18:01 — with Render Destroyed
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 14.28571% with 30 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop@0812f0a). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/resolvers/userResolver.ts 11.53% 23 Missing ⚠️
src/resolvers/profileResolver.ts 22.22% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #439   +/-   ##
==========================================
  Coverage           ?   31.17%           
==========================================
  Files              ?       87           
  Lines              ?     4225           
  Branches           ?     1234           
==========================================
  Hits               ?     1317           
  Misses             ?     2876           
  Partials           ?       32           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@Calebgisa72 Calebgisa72 added Ready for merge The pull request has received all necessary reviews and can be merged and removed Needs Reviews A pull request need reviews to be merged labels Nov 25, 2024
@Salim-54 Salim-54 merged commit d0413d0 into develop Nov 25, 2024
2 checks passed
Copy link

@RWEMAREMY RWEMAREMY left a comment

Choose a reason for hiding this comment

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

Good job

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready for merge The pull request has received all necessary reviews and can be merged steppers Steppers Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants