Context not found error

Question

Context not found error

Ik-williams opened this issue 7 years ago · 10 comments

After following all the necessary steps to install and run the app. When trying to access the app on localhost:3000 it says context not found. The live app seems to be running purely on react-js and redux without nextjs.

Answer 1 · 2018-12-08T23:33:23.000Z

Hello @Ik-williams Can you check whether this PR https://github.com/taming-the-state-in-react/nextjs-redux-firebase-authentication/pulls works for you by checking out the branch? If you confirm, then I can merge the Pull Request. Had no time to do this myself, sorry.

Answer 2 · 2018-12-09T12:06:22.000Z

Hello mate!! Thanks for the quick response. I have checked the pull request from https://github.com/taming-the-state-in-react/nextjs-redux-firebase-authentication/pulls and it woks fine. But there is one slight issue when saving the new user. The user is saved as undefined in firebase. The issue is that the new sign up details are inserted within users collection inside an object undefined and when queried returns undefined as an object with user details. Im sure there is an easy fix for this. Thanks alot tho !!

…

On Sun, 9 Dec 2018 at 00:33, Robin Wieruch ***@***.***> wrote: Hello @Ik-williams <https://github.com/Ik-williams> Can you check whether this PR https://github.com/taming-the-state-in-react/nextjs-redux-firebase-authentication/pulls works for you by checking out the branch? If you confirm, then I can merge the Pull Request. Had no time to do this myself, sorry. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AV68raD6nPB4ysgxONRClfKoUQdofYouks5u3ExEgaJpZM4ZJwVC> .

Answer 3 · 2018-12-11T02:43:40.000Z

Anyway you are able to add the Fix to the PR? Then we could merge it and people would be happy to have a version without bugs.

Answer 4 · 2018-12-11T12:41:14.000Z

Done!! created the pull request as requested.

Answer 5 · 2018-12-11T13:25:32.000Z

Thank you @Ik-williams 👍 But I hoped you would extends the PR of @leonardoballand Now we have two Pull Requests for this thing 😬

#7
#4

@Ik-williams @leonardoballand how do we proceed? 🎉

Answer 6 · 2018-12-11T13:49:43.000Z

He needs to commit on my PR. Or you could delete mine and merge the new PR if it is based on mine. Did you already review it?

Answer 7 · 2018-12-11T16:17:59.000Z

rwieruch, bruh Just delete one of the PR. @leonardoballand is correct.

Answer 8 · 2018-12-12T01:48:57.000Z

Which one should I merge? :)

Answer 9 · 2019-01-16T08:30:11.000Z

I can suggest #4 as it was more stable when I pulled it down.

Answer 10 · 2019-04-28T16:16:12.000Z

Fixed in #7