Skip to content

Comments

Ship prebuilt JavaScript with the cable_ready gem#191

Merged
marcoroth merged 13 commits intomasterfrom
ship-prebuilt-js-with-gem
Apr 1, 2022
Merged

Ship prebuilt JavaScript with the cable_ready gem#191
marcoroth merged 13 commits intomasterfrom
ship-prebuilt-js-with-gem

Conversation

@marcoroth
Copy link
Member

@marcoroth marcoroth commented Mar 17, 2022

Type of PR

Feature

Description

This enables Rails Applications to easily include the CableReady JavaScript client into their app by just installing the cable_ready Rubygem.

# Gemfile

gem 'cable_ready', '~> 5.0'

Either with Import maps:

# config/importmap.rb

# ...

pin 'cable_ready', to: 'cable_ready.min.js', preload: true

Or with the regular Asset pipeline:

<!-- app/views/layouts/application.html.erb -->

<%= javascript_include_tag "cable_ready.umd.min.js", "data-turbo-track": "reload" %>

Why should this be added

To get your Non-webpack/non-esbuild powered Rails 7 Application up and running with CableReady by just installing the Rubygem.

Checklist

  • My code follows the style guidelines of this project
  • Checks (StandardRB & Prettier-Standard) are passing
  • This is not a documentation update

@marcoroth marcoroth force-pushed the ship-prebuilt-js-with-gem branch from 45daf95 to 9c59654 Compare March 17, 2022 09:33
Copy link
Contributor

@julianrubisch julianrubisch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm very excited for this!

@marcoroth marcoroth force-pushed the ship-prebuilt-js-with-gem branch from 1ed7699 to 2bc4e9d Compare March 17, 2022 10:19
@marcoroth marcoroth force-pushed the ship-prebuilt-js-with-gem branch from 1694f1c to 3351dc9 Compare March 18, 2022 09:57
@leastbad
Copy link
Contributor

I am highly appreciative of all of the work happening in this PR.

@marcoroth marcoroth merged commit a30b4e8 into master Apr 1, 2022
@marcoroth marcoroth deleted the ship-prebuilt-js-with-gem branch April 1, 2022 15:55
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.

3 participants