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

Fixed Service Worker #976

Merged
merged 4 commits into from
Apr 15, 2019
Merged

Fixed Service Worker #976

merged 4 commits into from
Apr 15, 2019

Conversation

harshkhandeparkar
Copy link
Member

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!

  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with npm test
  • code is in uniquely-named feature branch and has no merge conflicts
  • PR is descriptively titled
  • ask @publiclab/is-reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If 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!

@harshkhandeparkar
Copy link
Member Author

@subhahu123 @publiclab/is-reviewers

@aashna27
Copy link

aashna27 commented Apr 2, 2019

Sorry to say but I tried it locally, the changes don't work.

@harshkhandeparkar
Copy link
Member Author

What is the error shown?

@aashna27
Copy link

aashna27 commented Apr 2, 2019

What is the error shown?

image

@harshkhandeparkar
Copy link
Member Author

So I think the service worker has to be in exmaples/ right?

@harshkhandeparkar
Copy link
Member Author

Ok I'll fix that. Wait a min.

@subhahu123
Copy link

Ok I'll fix that. Wait a min.

Can u provide the preview link also
@harshkhandeparkar

@harshkhandeparkar
Copy link
Member Author

Try it out locally now.
@aashna27 @subhahu123

@subhahu123
Copy link

Try it out locally now.
@aashna27 @subhahu123

@harshkhandeparkar i think this repo really needs deploy links which automatically generates pr

@harshkhandeparkar
Copy link
Member Author

You are working on it right?

@harshkhandeparkar
Copy link
Member Author

Surge?

@harshithpabbati
Copy link

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.
https://stackblitz.com/docs

@subhahu123
Copy link

@harshithpabbati @harshkhandeparkar we can go for both

@subhahu123
Copy link

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.
https://stackblitz.com/docs

@harshithpabbati i have use it various time i dont think so that it allows importing projects from gitthub

@subhahu123
Copy link

Surge?

@harshkhandeparkar can u work on that please i m working on my gsoc projects

@harshkhandeparkar
Copy link
Member Author

I have exams and I have no experience with it. Sorry.

@harshithpabbati
Copy link

If you are busy @harshkhandeparkar then I would like to do that.

@subhahu123
Copy link

I have exams and I have no experience with it. Sorry.

me also

@subhahu123
Copy link

If you are busy @harshkhandeparkar then I would like to do that.

please go for it @harshithpabbati

@harshkhandeparkar
Copy link
Member Author

Good luck !

@harshithpabbati
Copy link

But the case is i don't have experience with it. Fine i will do it somehow.

@subhahu123
Copy link

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

@subhahu123
Copy link

But the case is i don't have experience with it. Fine i will do it somehow.

Knowing about the building or compiling of code is enough

@harshkhandeparkar
Copy link
Member Author

But the case is i don't have experience with it. Fine i will do it somehow.

Np, you will only get to learn something new. If at all you get stuck, we are here to help!

@subhahu123
Copy link

But the case is i don't have experience with it. Fine i will do it somehow.

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
it is only about three commands and u r done

@harshkhandeparkar
Copy link
Member Author

I am using my phone so I cannot do anything. Srry.

@harshithpabbati
Copy link

Was surge implemented anywhere???

@subhahu123
Copy link

subhahu123 commented Apr 2, 2019

Was surge implemented anywhere???

community toolbox pr
working properly with no error
publiclab/community-toolbox#147
@harshithpabbati

@harshithpabbati
Copy link

Fine cool!

@harshkhandeparkar
Copy link
Member Author

@jywarren this is a small change. Try to merge it.

@aashna27
Copy link

aashna27 commented Apr 8, 2019

I would like to work on surge, is it available?

@harshithpabbati
Copy link

I did half of it. You can continue it.
Thanks.

@aashna27
Copy link

aashna27 commented Apr 8, 2019

Okay

@aashna27
Copy link

aashna27 commented Apr 8, 2019

So you create your pr, and then I ll commit my changes and we can get them merged.

@harshithpabbati
Copy link

harshithpabbati commented Apr 8, 2019

I already opened a pr. #982

@harshkhandeparkar
Copy link
Member Author

@jywarren could you please merge this?

@jywarren
Copy link
Member

Ok!!

@jywarren jywarren merged commit 2f86dcb into main Apr 15, 2019
@harshkhandeparkar
Copy link
Member Author

Thanks a lot!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Service Worker is not working properly
5 participants