Skip to content

feat(pin): taper son code avec le clavier#113

Open
Ciflire wants to merge 9 commits intoCETEN-OpenBar:mainfrom
Ciflire:enter_pin_with_kbd
Open

feat(pin): taper son code avec le clavier#113
Ciflire wants to merge 9 commits intoCETEN-OpenBar:mainfrom
Ciflire:enter_pin_with_kbd

Conversation

@Ciflire
Copy link
Contributor

@Ciflire Ciflire commented Oct 3, 2024

Fix #111
Doit être merge APRES #112

@yyewolf yyewolf assigned aripot007 and Ciflire and unassigned aripot007 Oct 6, 2024
@Ciflire Ciflire marked this pull request as draft October 7, 2024 15:48
@Ciflire
Copy link
Contributor Author

Ciflire commented Oct 7, 2024

je change en draft pour plusieurs raisons, la première c'est que je dois check quelques trucs avant le merge, la deuxième c'est qu'il y a deux components pour le pin, qui interviennent à différents endroits, je pense faire un refactor au passage de ce component

@Ciflire Ciflire marked this pull request as ready for review October 7, 2024 19:08
@Ciflire
Copy link
Contributor Author

Ciflire commented Oct 7, 2024

ok on va conserver ça de cette façon avec ce fix, à noter que ça ne marchera pas partout, mais ça marchera pour le client

La raison est que le components pin.svelte a été dupliqué, il en existe deux versions dans la codebase utilisées à différents endroits, cette PR est une version temporaire, j'envisage d'en faire une suivante avec un refactor du front et notamment des composants qui sont réutilisés, à l'heure actuelle je ne me vois pas le faire de peur d'oublier de modifier un composant et de tout casser

Copy link
Contributor

@aripot007 aripot007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça marche sur /borne mais les changements ont pas été fait sur /client

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.

[FR] Entrer le code avec le clavier

3 participants