Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch from jquery_ujs to rails-ujs #3390

Merged
merged 2 commits into from
Sep 18, 2021
Merged

Conversation

jdufresne
Copy link
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
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