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 missing pwa plugin #343

Closed
wants to merge 1 commit into from
Closed

fix: Add missing pwa plugin #343

wants to merge 1 commit into from

Conversation

BekahHW
Copy link
Member

@BekahHW BekahHW commented Jun 24, 2024

Description

The build is still currently breaking. One of the problems is we don't have the pwa plugin installed. This PR installs the plugin and the build is working locally.

Related Tickets & Documents

closes #342

Mobile & Desktop Screenshots/Recordings

Steps to QA

Tier (staff will fill in)

  • Tier 1
  • Tier 2
  • Tier 3
  • Tier 4

[optional] What gif best describes this PR or how it makes you feel?

@BekahHW BekahHW requested a review from a team as a code owner June 24, 2024 19:14
@BekahHW BekahHW requested a review from a team June 24, 2024 19:14
Copy link

netlify bot commented Jun 24, 2024

Deploy Preview for saucedocs ready!

Name Link
🔨 Latest commit 460e542
🔍 Latest deploy log https://app.netlify.com/sites/saucedocs/deploys/6679c594892cff00088847ac
😎 Deploy Preview https://deploy-preview-343--saucedocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@BekahHW BekahHW removed the request for review from a team June 24, 2024 19:28
@@ -3,7 +3,7 @@
"version": "1.75.4",
"private": true,
"engines": {
"node": "^18.17.0 || >=20.5.0",
"node": "^18.17.0 || >=20.5.0",
Copy link
Member

Choose a reason for hiding this comment

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

Was there a missing commit? This doesn't look to have added anything, just a formatting / spacing change to the package.json file

@BekahHW BekahHW closed this Jun 27, 2024
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.

bug: Add pwa plugin
2 participants