Skip to content

Conversation

@Xantass
Copy link
Contributor

@Xantass Xantass commented Jun 27, 2025

Describe your changes

add new route for get data of orders send

How to test the feature

call the route 'ordersCount' and check the return data

Issue ticket number and link

Closes #173

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 27, 2025 22:01
@Xantass Xantass self-assigned this Jun 27, 2025
@Xantass Xantass linked an issue Jun 27, 2025 that may be closed by this pull request
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.

You can't use clientCounts because it only counts 'Sur place' orders, in this case we want to be able to display the number of orders whatever their channel
+ missing documentation and confilct

@Xantass Xantass requested a review from JulesGresset June 28, 2025 10:22
@JulesGresset JulesGresset merged commit 4fd86cf into dev Jun 28, 2025
8 checks passed
@JulesGresset JulesGresset deleted the 173-route-for-kds-graph branch June 28, 2025 19:57
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 for KDS graph

3 participants