feat(comptoir): réinitialise le champ de saisie une fois la commande validée

[Ciflire]

solves #107

J'ai appris en resolvant cette issue que toute variable/fonction partagée doit être mise dans une forme de store, ensuite on fait souscrire nos components à cette variable

Donc il y a une variable dans le store, je l'extrais dans transactions.svelte pour faire la requête et j'associe sa valeur au champ de saisie pour qu'elle soit bien réinitialisée à la validation de la commande

Testé de mon coté ça m'a l'air bon, je ne l'ai fait que pour la validation de commande réussie, n'hésitez pas à vous mettre d'accord au bar à quel moment vous souhaitez le réinitialiser, de toutes façons tout est en place pour qu'il n'y ait qu'à rajotuer une ligne

[Ciflire]

ah j'en ai profité pour bump les versions des packages de dev et des modules npm, un svelte-check n'a rien révélé

[Ciflire]

Je ne vois pas ce que tu veux dire du tout, tu veux dire qu'il faut qu'il pull à nouveau à chaque clear du champ les nouvelles transactions, donc le forcer à se mettre à jour plus souvent que toutes les deux secondes?

[aripot007]

Yes exactement, c'est ce qui est fait actuellement, quand on entre quelque chose dans la barre de recherche ça met à jour les transactions directement

[Ciflire]

c'est toujous censé le faire, ça reload on:type cf https://github.com/CETEN-OpenBar/bar/pull/112/files?diff=split&w=0#diff-e4ab54ab30c3e70b18e2b3930068a985fd233414a4967d89b411fc337961cb0fR103

[aripot007]

L'event doit pas être déclenché quand la modif de la valeur est faite avec un bind, si tu veux bien le voir change juste le délai (l. 40 dans transactions.svelte), faut juste rajouter un reloadTransactions() quand tu subscribe à ton store

[Ciflire]

faudrait juste voir pour que les transactions soient reload dès que la barre de recherche est clear (pour le moment ça le fait avec le reload automatique toutes les 2 secondes)

ça le fait à chaque caractère donc à forciori ça le fait quand la barre est clear

Yes exactement, c'est ce qui est fait actuellement, quand on entre quelque chose dans la barre de recherche ça met à jour les transactions directement

ça n'entre pas en contradiction avec ce qui est fait

L'event doit pas être déclenché quand la modif de la valeur est faite avec un bind, si tu veux bien le voir change juste le délai (l. 40 dans transactions.svelte), faut juste rajouter un reloadTransactions() quand tu subscribe à ton store

il est bien déclenché j'ai fait un simple

                <input 
                    class="rounded-lg p-2 text-black"
                    placeholder="Rechercher une personne"
                    bind:value={$searchName}
                    on:input={(e) => {
                        // @ts-ignore
                        searchNameValue = e.target.value.toLowerCase();
                        console.log(searchNameValue)
                        page = 1;
                        reloadTransactions();
                    }}
                />

pour observer que c'était bien reload sur chaque char

[aripot007]

My bad j'aurais du préciser, je parle de quand tu clear la barre à la validation de la transaction en changeant directement la valeur du store depuistransactionPopup

Pour reproduire :

1. Augmenter le délai l.40 dans transactions.svelte (par ex mettre 2000000000)

2. Passer deux commandes avec deux comptes différents

3. Chercher le nom d'un des deux comptes et valider sa commande

4. La barre de recherche est réinitialisée, mais les transactions ne sont pas reload

[Ciflire]

Je vais le faire mais je ne suis pas certain que ça ait déjà été le cas, lorsqu'on finalise une transaction il n'y a jamais eu de reload

[Ciflire]

c'est fix du coup