Skip to content

Conversation

@Xantass
Copy link
Contributor

@Xantass Xantass commented Jun 26, 2025

Describe your changes

add new route for get all KPI for POS or KDS

How to test the feature

call the route 'displayKpi' and check the data return

Issue ticket number and link

Closes #172

Checklist before requesting a review

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have performed a self-review of my code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Documentation has been updated as required
    If a single item in this checklist is not checked, the pull request cannot be accepted

@Xantass Xantass requested a review from JulesGresset June 26, 2025 12:24
@Xantass Xantass self-assigned this Jun 26, 2025
@Xantass Xantass linked an issue Jun 26, 2025 that may be closed by this pull request
@Xantass Xantass changed the title feat: KPI for client Jun 26, 2025
Copy link
Member

@JulesGresset JulesGresset left a comment

Choose a reason for hiding this comment

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

Globaly works but some issues :

  • averagePrepTime and averageWaitingTime are inverted in both KDS and POS
  • clientsCount does not work in both KDS and POS
  • ordersInProgress in POS doesnt works
  • Some console.log still here

@Xantass Xantass requested a review from JulesGresset June 27, 2025 08:54
@Xantass Xantass enabled auto-merge June 27, 2025 09:28
@Xantass Xantass merged commit 5d177c6 into dev Jun 27, 2025
8 checks passed
@Xantass Xantass deleted the 172-route-to-get-needed-kpi-for-clients branch June 27, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Route to get needed KPI for clients

3 participants