When updating the regex for determing allowed hostnames I had to save it for the code in the textarea below would update and it struck me that it would be wise to use javascript to update in realtime. Granted, it's not important but if you decide not to implement it, put a line that tells the user to save the configuration in order to update the config code as it may not be obvious to everyone.

Thanks for a great module btw!

Comments

elmuerte’s picture

elmuerte commented
Status: Active » Closed (won't fix)

The problem with that is that you don't have a "backup" of your settings, you might forget to press the save button. Also this way you get a safe fallback in case the throttle fails for whatever reason.

solipsist’s picture

solipsist commented

Okay then well in that case make it clear to people that they need to save it in order to see the changes take effect. It's probably not obvious to everyone.