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

feat: add Vite variable for determining service worker build context #20470

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

krissvaa
Copy link

Hilla needs to dynamically import modules/libraries for the service workers, so there is some property to determine if the build is for SW or base.

Fixes vaadin/hilla#2867

Copy link

sonarcloud bot commented Nov 14, 2024

Copy link

Test Results

1 146 files  ± 0  1 146 suites  ±0   1h 32m 17s ⏱️ - 1m 16s
7 489 tests ± 0  7 438 ✅  -  1  50 💤 ±0  0 ❌ ±0  1 🔥 +1 
7 842 runs   - 17  7 781 ✅  - 18  60 💤 ±0  1 ❌ +1 

For more details on these errors, see this check.

Results for commit 2cb831a. ± Comparison against base commit 05f0377.

@mshabarov mshabarov added the Contribution PRs coming from the community or external to the team label Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution PRs coming from the community or external to the team +0.0.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PWA endpoint errors while called from SW
3 participants