-
Notifications
You must be signed in to change notification settings - Fork 210
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
Fixed Service Worker #976
Fixed Service Worker #976
Conversation
@subhahu123 @publiclab/is-reviewers |
Sorry to say but I tried it locally, the changes don't work. |
What is the error shown? |
So I think the service worker has to be in exmaples/ right? |
Ok I'll fix that. Wait a min. |
Can u provide the preview link also |
Try it out locally now. |
@harshkhandeparkar i think this repo really needs deploy links which automatically generates pr |
You are working on it right? |
Surge? |
We can use stackblitz for this. We can import code from github from a particular branch and host it we will be getting a link to it. Check this for reference. |
@harshithpabbati @harshkhandeparkar we can go for both |
@harshithpabbati i have use it various time i dont think so that it allows importing projects from gitthub |
@harshkhandeparkar can u work on that please i m working on my gsoc projects |
I have exams and I have no experience with it. Sorry. |
If you are busy @harshkhandeparkar then I would like to do that. |
me also |
please go for it @harshithpabbati |
Good luck ! |
But the case is i don't have experience with it. Fine i will do it somehow. |
i don't think so u require any experience with it u can refer to my PR for surge console command |
Knowing about the building or compiling of code is enough |
Np, you will only get to learn something new. If at all you get stuck, we are here to help! |
@harshkhandeparkar u can also provide a preview link for this pr using surge |
I am using my phone so I cannot do anything. Srry. |
Was surge implemented anywhere??? |
community toolbox pr |
Fine cool! |
@jywarren this is a small change. Try to merge it. |
I would like to work on surge, is it available? |
I did half of it. You can continue it. |
Okay |
So you create your pr, and then I ll commit my changes and we can get them merged. |
I already opened a pr. #982 |
@jywarren could you please merge this? |
Ok!! |
Thanks a lot!! |
Fixes #972
Fixed the broken service worker
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
npm test
@publiclab/is-reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!