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

Dockerfile #682

Closed
wants to merge 3 commits into from
Closed

Dockerfile #682

wants to merge 3 commits into from

Conversation

sharkySharks
Copy link
Contributor

This PR is a:

  • New feature
  • Enhancement/Optimization
  • Refactor
  • Bugfix
  • Test for existing code
  • Documentation

Summary

When this pull request is merged, it will allow users to run PWA inside of a docker container. Users will also have the ability to mount their filesystem to the container and webpack hot reloading will continue to work with both changes on the host system as well as in the container.

@vercel
Copy link

vercel bot commented Dec 19, 2018

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

@PWAStudioBot
Copy link
Contributor

Fails
🚫

The following file(s) were not formatted with prettier. Make sure to execute npm run prettier locally prior to committing.

packages/pwa-buildpack/src/WebpackTools/PWADevServer.js
packages/venia-concept/webpack.config.js

Generated by 🚫 dangerJS

@coveralls
Copy link

coveralls commented Dec 19, 2018

Coverage Status

Coverage increased (+0.02%) to 61.441% when pulling cb437c4 on dockerfile into a15818f on release/2.0.

MAGENTO_BACKEND_URL="https://release-dev-rxvv2iq-zddsyhrdimyra.us-4.magentosite.cloud/"
MAGENTO_BUILDPACK_PROVIDE_SECURE_HOST=0
PWA_STUDIO_PORTS_DEVELOPMENT=8080
ENABLE_SERVICE_WORKER_DEBUGGING=1
Copy link
Contributor Author

@sharkySharks sharkySharks Dec 20, 2018

Choose a reason for hiding this comment

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

how do we get service workers running on secure hosting from a docker container.

@sharkySharks
Copy link
Contributor Author

work is in progress and will be updated here soon.

tracking this under #734

@sharkySharks sharkySharks added the hold On hold until another condition is fulfilled. label Jan 9, 2019
@sharkySharks
Copy link
Contributor Author

adding hold label until I update work here with https support

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.

5 participants