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

[PWA-324] [bug]: venia-static files don't load out-of-the box with scaffolding command #2143

Merged
merged 3 commits into from
Feb 10, 2020

Conversation

tjwiebell
Copy link
Contributor

@tjwiebell tjwiebell commented Feb 5, 2020

Description

See #2109

This fix resolves two issues, but has uncovered one more, which will be investigated and resolved in a fast follow.

Fixes:

  • 500 error of venia static assets when doing yarn watch or yarn start
  • venia-static missing from published package

Known Issues:

  • While using yarn watch, webpack is not actually serving the actual assets. Nothing is throwing an error, but something is wrong with the content type, or the headers, or something odd that webpack is doing with its virtual file system. You can navigate directly to assets and they load fine (eg. /venia-static/favicon.ico), so this should just be something simple with webpack or UPWARD. To be fixed by https://jira.corp.magento.com/browse/PWA-357.

Related Issue

Acceptance

Verification Stakeholders

Specification

Verification Steps

Note: The DEBUG_PROJECT_CREATION flag below will simulate fetches of package tarballs from npm by running npm pack locally and serving those.

  1. From a sibling directory to pwa-studio, run the create-pwa binary (mkdir create-pwa-test; cd create-pwa-test; DEBUG_PROJECT_CREATION=1 ../pwa-studio/packages/create-pwa/bin/create-pwa
  2. Step through setup and install package dependencies
  3. yarn watch, and verify that 500 errors are no longer being thrown
  4. Optionally, you can also go into node_modules/@magento/venia-ui/ and verify that venia-static is now there in the package

Screenshots / Screen Captures (if appropriate)

Checklist

  • I have updated the documentation accordingly, if necessary.
  • I have added tests to cover my changes, if necessary.

@tjwiebell tjwiebell added the version: Patch This changeset includes backwards compatible bug fixes. label Feb 5, 2020
@m2-community-project m2-community-project bot added this to Ready for Review in Pull Request Progress Feb 5, 2020
@PWAStudioBot
Copy link
Contributor

PWAStudioBot commented Feb 5, 2020

Messages
📖

Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.

📖 DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next nightly build run (assuming they are fixed).

Generated by 🚫 dangerJS against 2b8ee2e

Copy link
Contributor

@revanth0212 revanth0212 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While reviewing this I noticed another issue with create-pwa tool. If in the install step you give the answer as n, the file paths for local repos in package.json are wrong. I will create a new bug for that. As for this bug, it is fixed.

@m2-community-project m2-community-project bot moved this from Ready for Review to Reviewer Approved in Pull Request Progress Feb 7, 2020
@dpatil-magento dpatil-magento merged commit a5e5549 into develop Feb 10, 2020
@dpatil-magento dpatil-magento deleted the tommy/create-pwa-assets branch February 10, 2020 19:35
@m2-community-project m2-community-project bot moved this from Reviewer Approved to Done in Pull Request Progress Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:venia-ui version: Patch This changeset includes backwards compatible bug fixes.
Development

Successfully merging this pull request may close these issues.

[bug]: venia-static files don't load out-of-the box with scaffolding command
4 participants