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

Updated when the the javascript function runs. #15

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

Conversation

JessSumner
Copy link

Why:
I found when implementing the gem that on a page refresh the dependent fields were applied. However when the route to the new form was followed the dependency was not functioning.

This PR:
Adds to the code instructions that on every page load that the function should occur. Adding this to my project means that whether the page is refreshed or loaded having followed its route the dependent fields display correctly.

Why:
I found when implementing the gem that on a page refresh the dependent fields were applied. However when the route to the new form was followed the dependency was not functioning.

This PR:
Adds to the code instructions that on every page load that the function should occur. Adding this to my project means that whether the page is refreshed or loaded having followed its route the dependent fields display correctly.
Why:
On further understanding of the issue I realize that my initial commit was not completely accurate.

This commit:
Includes the original jQuery and my updated jQuery. The original code will work if that app does not have Turbolinks included. If Turbolinks is active in the app my updated jQuery is necessary for the function to run on every page load.
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.

1 participant