Sollicitation : trouver son siret par le nom de son entreprise #1376
Conversation
There was a problem hiding this comment.
Ça m’a l’air cool à part ça. J’ai l’impression qu’il se passe des trucs bizarres quand on lance une recherche avec un seul caractère, une requête est faite mais on n’affiche rien. Peut-être qu’en plus du debounce, on devrait de toute façon éviter de faire une requête s’il y a moins de trois caractères.
| } | ||
|
|
||
| function isSiretSearch(str) { | ||
| return str.match(/^\d+$/g); |
There was a problem hiding this comment.
Comme on disait ce matin, on veut aussi tolérer les espaces dans le siret
There was a problem hiding this comment.
oui, c'est bien pris en compte à présent
|
Tu n'affiches pas l'adresse de l'entreprise finalement ? |
|
... vous êtes en train de regarder un ancien commit... |
|
Je comprends pas comment je peux avoir des conflits avec ma propre branche, bon sang |
clairezed
force-pushed
the
find_siret_with_name
branch
from
6101819
to
1da433f
Compare
| .autocomplete__hint | ||
| -webkit-appearance: none | ||
| border: 1px solid #adb9c9 | ||
| border: 1px solid var(--theme-border) |
There was a problem hiding this comment.
C'est pas lié uniquement à cette PR mais je me demande parfois pourquoi on s'ennuie à mettre des variables qui ne sont pas reconnu par certains navigateurs que l'on doit doubler avec un code couleur, en cas de changement de couleur on doit repasser partout quand même.
There was a problem hiding this comment.
C'est vrai. C'est en attente que tous les navigateurs reconnaissent les variables css, quoi. Y'a pas un polyfil qui existerait ?
There was a problem hiding this comment.
ah purée, bien vu |
fixes #1247