Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
T
third-party
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 2
    • Merge Requests 2
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • cooperatic-foodcoops
  • third-party
  • Merge Requests
  • !171

Merged
Opened May 04, 2022 by François C.@fracolo 
  • Report abuse
Report abuse

Ticket 2828

  • Discussion 2
  • Commits 4
  • Pipelines 4
  • Changes 3
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • François C. @fracolo

    added 11 commits

    • ae224347...acd357d1 - 10 commits from branch dev_cooperatic
    • 0d39f02b - Merge branch 'dev_cooperatic' into 'ticket_2828'

    Compare with previous version

    May 04, 2022

    added 11 commits

    • ae224347...acd357d1 - 10 commits from branch dev_cooperatic
    • 0d39f02b - Merge branch 'dev_cooperatic' into 'ticket_2828'

    Compare with previous version

    added 11 commits <ul><li>ae224347...acd357d1 - 10 commits from branch <code>dev_cooperatic</code></li><li>0d39f02b - Merge branch &#39;dev_cooperatic&#39; into &#39;ticket_2828&#39;</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/171/diffs?diff_id=1617&start_sha=ae2243477de2de96456f5df2293d75dbac7d9a5c)
    Toggle commit list
  • Damien Moulard @DamienM commented May 05, 2022
    Developer

    Problèmes détectés :

    • On ne peut faire qu'un retour avec la flèche enroulée, il faudrait que le signe disparraisse après avoir cliqué dessus une fois.
    • Si je bip 2 fois, je fais retour, et je re-bip, ça me compte 3 fois la valeur au lieu de 2 (la valeur avant qu'on fasse retour est conservée)
    Problèmes détectés : * On ne peut faire qu'un retour avec la flèche enroulée, il faudrait que le signe disparraisse après avoir cliqué dessus une fois. * Si je bip 2 fois, je fais retour, et je re-bip, ça me compte 3 fois la valeur au lieu de 2 (la valeur avant qu'on fasse retour est conservée)
  • François C. @fracolo commented May 05, 2022
    Master

    En faisant les modifications et tests pour résoudre les 2 points soulevés, il apparait un cas d'utilisation non géré :

    • Si après une série de bip avec code-barre je corrige le total affiché (par exemple si un code-barre est illisible, ou si l'article avait déjà été compté et qu'un nouveau est trouvé), et qu'ensuite je rescanne un article, la modification faite n'est pas prise en compte.
    En faisant les modifications et tests pour résoudre les 2 points soulevés, il apparait un cas d'utilisation non géré : * Si après une série de bip avec code-barre je corrige le total affiché (par exemple si un code-barre est illisible, ou si l'article avait déjà été compté et qu'un nouveau est trouvé), et qu'ensuite je rescanne un article, la modification faite n'est pas prise en compte.
  • François C. @fracolo

    added 1 commit

    • 7dfcbf2f - Adaptations suite remarques PR 171

    Compare with previous version

    May 05, 2022

    added 1 commit

    • 7dfcbf2f - Adaptations suite remarques PR 171

    Compare with previous version

    added 1 commit <ul><li>7dfcbf2f - Adaptations suite remarques PR 171</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/171/diffs?diff_id=1623&start_sha=0d39f02b9b5d4d7b90e1a8bbd2b6ade4ef35b81e)
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    May 05, 2022

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit a5931d7c

    May 05, 2022

    mentioned in commit a5931d7c

    mentioned in commit a5931d7cdf44979bce17791472d5eb7f8626a9c5
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: cooperatic-foodcoops/third-party!171
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.