pxlrbt / filament-excel

Excel Export for Filament Admin Resources
MIT License
323 stars 68 forks source link

Adding new interaction with a confirmation message #153

Closed harrisonlucaswork closed 7 months ago

harrisonlucaswork commented 7 months ago

My support team keeps mashing the export button and its an expensive query. I added a confirmation message so I could put the help text in there to ask them not to click it so often and thought other people might enjoy it.

image
pxlrbt commented 7 months ago

I wouldn't expect a checkbox when using requiresConfirmation() as it's not the same behaviour that Filament has. You could also use requiresConfirmation() on the action itself if you don't use AskForFilename etc.

I think I'd rather add this as an example to a "cookbook" in the docs, than as a feature.