Fix @rails/ujs init / use ES6 module syntax

aaronskiba · aaronskiba · commit 5d06d6b86f6f · 2024-07-18T15:40:04.000-06:00
Prior to this code change, any value assigned to the `'data-method':` attribute of the `link_to` method was not being read (and instead defaulting to `GET`). This was resulting in the breaking of several `spec/features/` tests (https://github.com/DMPRoadmap/roadmap/actions/runs/9946998559/job/27478725801). The `@rails/ujs` library is meant to handle this `'data-method':` attribute.
diff --git a/app/javascript/application.js b/app/javascript/application.js
@@ -81,10 +81,11 @@ import './src/superAdmin/notifications/edit';
 import './src/superAdmin/themes/newEdit';
 import './src/superAdmin/users/edit';
 
-// Since we're using Webpacker to manage JS we need to startup Rails' Unobtrusive JS
-// and Turbo. ActiveStorage and ActionCable would also need to be in here
-// if we decide to implement either before Rails 6
-require('@rails/ujs').start();
+// We need to startup Rails' Unobtrusive JS and Turbo.
+// ActiveStorage and ActionCable would also need
+// to be in here if we decide to implement either.
+import Rails from "@rails/ujs";
+Rails.start();
 
 // TODO: Disabled turbo for the time being because our custom JS is not
 //       properly setup to work with it. We should review the docs:
