‘delete user self max post count’ n'accepte pas ‘-1’ pour désactiver

martin · Novembre 16, 2023, 2:12

La correction pour cela sera ici, elle devrait être fusionnée plus tard aujourd’hui :

github.com/discourse/discourse

FIX: min/max not passed to NumberField for site settings

main ← dev/fix-min-max-for-number-field

opened 02:08AM - 16 Nov 23 UTC

martin-brennan

+109 -5

When we started using NumberField for integer site settings in e113eff6634c9b39…8b0541ea6a0d1626f6732dcc, we did not end up passing down a min/max value for the integer to the field, which meant that for some fields where negative numbers were allowed we were not accepting that as valid input. This commit passes down the min/max options from the server for integer settings then in turn passes them down to NumberField. c.f. https://meta.discourse.org/t/delete-user-self-max-post-count-not-accepting-1-to-disable/285162

Sujet		Réponses	Vues
Thousand separator silently ruins inputted value in settings Bug fixed	7	763	Juillet 8, 2024
SiteSetting numbers not properly validated Bug	9	1830	Avril 23, 2015
Site Setting Integer Validation Issue UX	2	681	Juin 23, 2020
Internal server error trying to access site_settings Bug	4	1386	Avril 23, 2015
How do I remove the option for users to delete themselves? Product Support	1	555	Janvier 27, 2023

‘delete user self max post count’ n'accepte pas ‘-1’ pour désactiver

Sujets connexes