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

Manifest file should not receive %PUBLIC_URL% input when working with CRA #782

Open
dantman opened this issue Jan 19, 2022 · 2 comments
Open
Labels
bug Something isn't working

Comments

@dantman
Copy link

dantman commented Jan 19, 2022

Describe the bug

The README example includes --path "%PUBLIC_URL%" which I expect is for CRA support. However when using this option manifest.json paths include %PUBLIC_URL%. However CRA does not replace %PUBLIC_URL% in manifest.json.

Expected behavior

manifest.json should not receive the %PUBLIC_URL% prefix from --path.

@dantman dantman added bug Something isn't working needs verification Bug needs to be verified with reproduction labels Jan 19, 2022
@onderceylan onderceylan removed the needs verification Bug needs to be verified with reproduction label Apr 18, 2022
@onderceylan onderceylan changed the title %PUBLIC_URL% example does not actually work in CRA Manifest file should not receive %PUBLIC_URL% input when working with CRA Apr 18, 2022
@onderceylan
Copy link
Collaborator

Hi @dantman, thanks for sharing your feedback and reporting the issue. I will look into it asap.

@az0uz
Copy link

az0uz commented Jun 17, 2022

As a temporary workaround, I use this in my package.json:

{
  ...
  "scripts": {
    ...
    "generate-assets": "yarn pwa-asset-generator [options] && sed -i '' 's/%PUBLIC_URL%\\///' public/manifest.json"
  },
  ...
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants