‘delete user self max post count’ non accetta ‘-1’ per disabilitare

martin · 16 Novembre 2023, 2:12am

La correzione per questo sarà qui, dovrebbe essere unita più tardi oggi:

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

Argomento		Risposte	Visualizzazioni
Thousand separator silently ruins inputted value in settings Bug fixed	7	764	Luglio 8, 2024
SiteSetting numbers not properly validated Bug	9	1830	Aprile 23, 2015
Site Setting Integer Validation Issue UX	2	682	Giugno 23, 2020
Internal server error trying to access site_settings Bug	4	1386	Aprile 23, 2015
How do I remove the option for users to delete themselves? Product Support	1	555	Gennaio 27, 2023

‘delete user self max post count’ non accetta ‘-1’ per disabilitare

Argomenti correlati