Skip to content

Documentation asset.md update: Add needed require.context to use pack helpers #1969

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

Merged
merged 2 commits into from
Mar 18, 2019

Conversation

klausbadelt
Copy link
Contributor

As discussed in #705, the documentation is missing the essential addition of require.context in your app js to make the asset_pack_path and image_pack_tag

As discussed in rails#705, the documentation is missing the essential addition of require.context in your app js to make the `asset_pack_path` and `image_pack_tag`
@klausbadelt
Copy link
Contributor Author

When/if #1964 will be merged, the reference to media/ prefix will have to change again. (I vote for #1964, keeping subfolder paths).

@guilleiguaran
Copy link
Member

@klausbadelt thank you very much!

I've merged #1964, can you update this to reflect the changes introduced with that PR and with 0b86cad ?.

@klausbadelt
Copy link
Contributor Author

@guilleiguaran: this reflects now #1964 (merged) and clarifies how 0b86cad works.

@gauravtiwari gauravtiwari merged commit 554221c into rails:master Mar 18, 2019
@gauravtiwari
Copy link
Member

Thanks :)

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