Skip to content

Switch from jquery_ujs to rails-ujs#3390

Merged
mshibuya merged 2 commits into
railsadminteam:masterfrom
jdufresne:ujs
Sep 18, 2021
Merged

Switch from jquery_ujs to rails-ujs#3390
mshibuya merged 2 commits into
railsadminteam:masterfrom
jdufresne:ujs

Conversation

@jdufresne

Copy link
Copy Markdown
Member

rails-ujs has been available in Rails itself since 5.1.0:
https://github.com/rails/rails/tree/main/actionview/app/assets/javascripts

It obsoletes jquery_ujs. From the docs:
https://github.com/rails/jquery-rails

If you are running Rails 5.1 and up, and if you have included //=
require rails-ujs, then jquery_ujs is not needed anymore. You can just
add:

//= require jquery

This is a step towards reducing jQuery usage while still allowing for
UJS features.

Refs #2893

jdufresne and others added 2 commits September 16, 2021 20:41
rails-ujs has been available in Rails itself since 5.1.0:
https://github.com/rails/rails/tree/main/actionview/app/assets/javascripts

It obsoletes jquery_ujs. From the docs:
https://github.com/rails/jquery-rails

> If you are running Rails 5.1 and up, and if you have included //=
> require rails-ujs, then jquery_ujs is not needed anymore. You can just
> add:

    //= require jquery

This is a step towards reducing jQuery usage while still allowing for
UJS features.

Refs railsadminteam#2893
@mshibuya mshibuya merged commit 419f3ea into railsadminteam:master Sep 18, 2021
@mshibuya

Copy link
Copy Markdown
Member

Excellent, thanks!

@jdufresne jdufresne deleted the ujs branch September 18, 2021 18:09
mshibuya added a commit that referenced this pull request Sep 20, 2021
mshibuya added a commit that referenced this pull request Sep 20, 2021
mshibuya added a commit that referenced this pull request Sep 21, 2021
mshibuya added a commit that referenced this pull request Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants