tailwindcss-stimulus-components icon indicating copy to clipboard operation
tailwindcss-stimulus-components copied to clipboard

Add Rails/ujs to Autosave

Open Merovex opened this issue 3 years ago • 1 comments

The current Autosave references Rails.fire but does not include Rails. Adding import Rails from '@rails/ujs' should address this.

Merovex avatar Jul 24 '22 12:07 Merovex

What is the consensus on using .requestSubmit() rather than Rails.fire?

This would make it compatible with Turbo and remove the dependency on Rails/ujs.

hcurotta avatar Aug 15 '22 12:08 hcurotta

This isn't using Rails.fire any more (11f7338).

Agree with @hcurotta that it should be switched to .requestSubmit though

jbennett avatar Jul 01 '23 13:07 jbennett