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: render custom offline page in app shell #10360
Conversation
bcff1d2
to
31716bd
Compare
31716bd
to
27ec0b0
Compare
27ec0b0
to
54e05eb
Compare
`/offline-stub.html` returns either user-provided (via @pwa(offlinePath) or the default generated offline page.
54e05eb
to
4843f63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good and works fine, great work.
A minor comment could be updated.
What's the breaking change? why use feat! instead of feat?
flow-server/src/main/java/com/vaadin/flow/server/PwaRegistry.java
Outdated
Show resolved
Hide resolved
Right, maybe |
Hi @joheriks , this commit cannot be picked to 6.0 by this bot, can you take a look and pick it manually? |
* feat!: serve `offline-stub.html` `/offline-stub.html` returns either user-provided (via @pwa(offlinePath) or the default generated offline page. * Comment fix * Use relative path ./ instead of resolver.baserUrl * fix tests Co-authored-by: haijian <haijian@vaadin.com>
* feat!: serve `offline-stub.html` `/offline-stub.html` returns either user-provided (via @pwa(offlinePath) or the default generated offline page. * Comment fix * Use relative path ./ instead of resolver.baserUrl * fix tests Co-authored-by: haijian <haijian@vaadin.com>
* feat!: serve `offline-stub.html` `/offline-stub.html` returns either user-provided (via @pwa(offlinePath) or the default generated offline page. * Comment fix * Use relative path ./ instead of resolver.baserUrl * fix tests Co-authored-by: haijian <haijian@vaadin.com>
Agreed in Slack that it's ok to release this as a patch for 6.0.x even though the commit has been marked as |
Fixes #10160.