Skip to content

Conversation

@Xantass
Copy link
Contributor

@Xantass Xantass commented Jun 22, 2025

Describe your changes

Use the new format of ingredient into route '/food/:id?useCase=POS'

How to test the feature

Call the route '/food/:id?useCase=POS' and check if the return was ok

Issue ticket number and link

Closes #164

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 22, 2025 10:50
@Xantass Xantass self-assigned this Jun 22, 2025
@Xantass Xantass linked an issue Jun 22, 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.

The new format cause an error in the POS when using with the useCase for POS

Error appears when clicking on the food and opening the modifications submenu
Capture d’écran 2025-06-22 à 14 39 25

@Xantass Xantass requested a review from JulesGresset June 23, 2025 09:01
@Xantass Xantass enabled auto-merge June 23, 2025 09:01
@JulesGresset JulesGresset force-pushed the 164-new-format-of-ingredients-for-food-routes branch from f3a5291 to 6c8dbbd Compare June 24, 2025 20:15
@Xantass Xantass merged commit 5da6e43 into dev Jun 24, 2025
15 checks passed
@Xantass Xantass deleted the 164-new-format-of-ingredients-for-food-routes branch June 24, 2025 20:17
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.

New format of ingredients for food routes

3 participants