search

enioka-Haute-Couture / enioka_scan

Integration library for all your Android hardware barcode scanner integration needs
Apache License 2.0
52 stars 4 forks source link

192 - Add scanner issues to docs #193

Closed VincentKobz closed 5 months ago

VincentKobz commented 5 months ago

https://github.com/enioka-Haute-Couture/enioka_scan/issues/192

Current know issues on scanners:

DaSpood commented 5 months ago

Pour tes PR ouvertes et les futures: attention a ne pas faire avoir de branchements en chaine, il faut penser a revenir sur une branche "saine" et ne pas partir de ta branche courante. Cette PR part de la #191 qui part elle meme de la #190, elle meme partant de la #186, alors qu'elles n'ont a priori rien en commun (c'etait comprehensible de partir de la branche rework de l'app demo, mais pas plus loin). Cela retarde les reviews car la diff est polluée.

Pour les PR 190-191-193: rebase interactif pour supprimer les commits qui ne sont pas en lien direct avec le sujet de la PR

VincentKobz commented 5 months ago

Pour tes PR ouvertes et les futures: attention a ne pas faire avoir de branchements en chaine, il faut penser a revenir sur une branche "saine" et ne pas partir de ta branche courante. Cette PR part de la #191 qui part elle meme de la #190, elle meme partant de la #186, alors qu'elles n'ont a priori rien en commun (c'etait comprehensible de partir de la branche rework de l'app demo, mais pas plus loin). Cela retarde les reviews car la diff est polluée.

Pour les PR 190-191-193: rebase interactif pour supprimer les commits qui ne sont pas en lien direct avec le sujet de la PR

Hello, c'est rebase ! Je ferais attention la prochaine fois a ne pas partir de la branche courante, merci pour les explications.