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

Login and signup with facebook and email working #173

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

varun1002
Copy link

Prerequisites

I am running the latest version

The login and sign up with email/password and facebook is not working but after using correct imports from firebase and using correct function from react-native-fbsdk it is working fine now

Copy link
Collaborator

@shehand shehand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the comment. Other changes are good to go. And make sure to add a working video of the change in the app.

@@ -10,7 +10,7 @@ import {
ScrollView,
} from "react-native";
import { AccessToken, LoginManager } from "react-native-fbsdk";
import { f, auth } from "../../../config/config.js";
import { f, auth } from "../../../config/config.example.js";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wrong. We should use config.js and not config.example.js.
The idea of having config.example.js is to give a template to the developer to setup the config file and the file should be renamed to config.js when doing the development. So please revert this change.

@varun1002
Copy link
Author

Before making changes

image

After using correct imports

WhatsApp.Video.2023-03-24.at.2.43.15.PM.mp4

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.

2 participants