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.

Sign up for GitHub

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

Jump to bottom

feat(workbox): add auto update to change the app content #481

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat(workbox): add auto update to change the app content #481

wants to merge 2 commits into from

Conversation

emechpi
Copy link

@emechpi emechpi commented Aug 28, 2021

Resolve #479

@emechpi emechpi changed the title Feat/auto update feat(workbox): add auto update to change the app content Aug 28, 2021
@manniL
Copy link
Member

manniL commented Sep 4, 2021

I think this code should live in the userland. Not every app might want this behavior.

@emechpi
Copy link
Author

emechpi commented Sep 22, 2021 •
edited

@manniL
But when we talk about the PWA applications, the auto-update of application content is one of the most important advantages of PWA apps. I think at least we can add a new property in module config and let developor to decide about it.
I can work on it :)

@steveDL
Copy link

steveDL commented Dec 9, 2021

Hey is this being merged in?

@Andrevoks
Copy link

why is this still not merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

the app Auto update dosen't work !
4 participants
@emechpi @manniL @steveDL @Andrevoks