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

fix: add template support for homepage, browserslist properties #8209

Closed
wants to merge 1 commit into from

Conversation

jimthedev
Copy link

This change updates the init script to allow for the homepage property to be set by a template. The homepage field with a value of "." is used when running applications at non-root of a domain. Additionally this adds support for templates to customize the browserlist property. This is useful specifically when a template includes IE support out of the box.

This change updates the init script to allow for the homepage property to be set by a template. The homepage field with a value of "." is used when running applications at non-root of a domain. Additionally this adds support for templates to customize the browserlist property. This is useful specifically when a template includes IE support out of the box.
@mrmckeb
Copy link
Contributor

mrmckeb commented Dec 19, 2019

Hi @jimthedev, this is a great fix - however, someone is working on adding support for all package.json keys already, and this approach is not compatible with the other.

#8082 (comment)

Please chime in on that thread and thanks for your time.

@mrmckeb mrmckeb closed this Dec 19, 2019
@mrmckeb
Copy link
Contributor

mrmckeb commented Dec 19, 2019

We would aim to get this out in 3.4 BTW, so it should not be too far away.

@lock lock bot locked and limited conversation to collaborators Dec 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants