Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[patch] Validate multi-url input #178

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

mguihal
Copy link
Collaborator

@mguihal mguihal commented Apr 20, 2024

Hello,

Il y a actuellement un bug sur les pages Organisation, quand on met une valeur qui n'est pas une url valide dans le champ "Site web".

J'ai changé le champ :

  • en tableau de champs car visiblement on pouvait auparavant mettre plusieurs sites web pour une organisation, donc un seul input c'est pas l'idéal
  • renforcé la validation de l'input avec un type="url" qui fera l'affaire pour l'instant

PR associée sur Semapps pour l'affichage de la page si de telles valeurs sont déjà saisies : assemblee-virtuelle/semapps#1245

@mguihal mguihal self-assigned this Apr 20, 2024
@mguihal mguihal force-pushed the fix-ValidateMultipleUrlInput branch from 3973cfc to d34049d Compare May 6, 2024 21:23
@mguihal mguihal force-pushed the fix-ValidateMultipleUrlInput branch from d34049d to 06733ee Compare July 7, 2024 21:10
@mguihal mguihal force-pushed the fix-ValidateMultipleUrlInput branch from 06733ee to bbd03d1 Compare July 14, 2024 22:04
@mguihal mguihal force-pushed the fix-ValidateMultipleUrlInput branch from bbd03d1 to a0825a9 Compare July 25, 2024 23:37
@mguihal mguihal merged commit aa94ede into master Jul 26, 2024
@mguihal mguihal deleted the fix-ValidateMultipleUrlInput branch July 26, 2024 21:33
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.

3 participants