-
Notifications
You must be signed in to change notification settings - Fork 27
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
Updates and clarifications to Readme + Offline app #8
Conversation
Note: We have done some prelim tests and it's working well, pls run your own tests too.
Please ignore the last commit (still working on some edge cases bugs) |
Interesting PR, thanks for your participation. Pertinent information have been added to the Readme. |
Okay so I'm just going to take the changes made to the readme.md as you still didn't fix the sw.js |
I am not sure I understand, I am looking at this thread and there is nothing to comment on. What you can do is take what you like from the PR and leave the rest out. Specifically, I am working on sw.js itself. But the readme can use the changes I propose. |
Okay I taught you wanted me to wait you to update the SW.js before doing anything about this PR. |
Okay I'm going to not merge this PR but just make the needed changes on my local Readme.md then |
Sorry for the delay I taught for a moment that I've updated the repo already. 8690c88 is based on your update on the Readme. |
No description provided.