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

Make it possible to use our own initialize and some other improvements #135

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jonathan-s
Copy link
Owner

@jonathan-s jonathan-s commented Jul 25, 2021

Type of PR (feature, enhancement, bug fix, etc.)

Description

A couple of improvements are introduced here.

  • Be able to pin what version of StimulusReflex we are using. The
    tradeoff is that we need to maintain a fork that is lightly touched
    to allow for that functionality.
  • Be able to decouple from how rails expect the identifier would look
    like
  • The javascript code for actioncable is no longer a dependency.

Checklist

  • Make sure that we have debug in the sockpuppet part.
  • Make an appropriate tag for the fork.
  • Tests are passing
  • Documentation has been added or amended for this feature / update

@jonathan-s jonathan-s added the enhancement New feature or request label Jul 25, 2021
@jonathan-s jonathan-s force-pushed the pinned-js branch 2 times, most recently from dcb85e2 to 9158107 Compare July 25, 2021 20:14
A couple of improvements are introduced here.
- Be able to pin what version of StimulusReflex we are using. The
tradeoff is that we need to maintain a fork that is lightly touched
to allow for that functionality.
- Be able to decouple from how rails expect the identifier would look
like
- The javascript code for actioncable is no longer a dependency.
@jonathan-s
Copy link
Owner Author

@DamnedScholar Perhaps you could have a look at this and let me know if you have any thoughts?

@DamnedScholar
Copy link
Contributor

Thank you. I will probe it when I get a chance.

@jonathan-s jonathan-s mentioned this pull request Aug 1, 2021
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants