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

User not getting pushed to Mailchimp when using Apple sign in provider #62

Open
ben-wright opened this issue Mar 15, 2023 · 6 comments
Open
Labels
bug Something isn't working needs author feedback Pending additional information from the author needs investigation Investigation needed to find more information, e.g. is this a bug, is the feature viable.

Comments

@ben-wright
Copy link

Hi,

The extension is working great when users signup using email and password provider. However, when user using Apple Sign In provider, they are not pushed out to mailchimp.

I have checked logs for ext-mailchimp-firebase-sync-addUserToList function and it seems it is not getting trigger when a user signs up using Apple.

Any help would be greatly appreciated, thanks!

@ben-wright ben-wright changed the title User not getting pushed to Mailchimp when Apple sign in provider is used User not getting pushed to Mailchimp when using Apple sign in provider Mar 15, 2023
@barticus
Copy link
Collaborator

Hi @ben-wright , sorry for the slow response on this one.
This sounds like odd behaviour.

  1. Are you able to confirm that you are seeing new users in Firebase Auth who have logged in with Apple?
  2. Can you confirm you're following roughly the process outlined here, particularly that you are using Auth.auth().signIn(with: credential) to sign the user in? This is mostly to check you're not potentially using a custom token which would not trigger the handler

Otherwise, I'll start trying to repro this, but it requires a few steps to get set up!

@barticus barticus added bug Something isn't working needs investigation Investigation needed to find more information, e.g. is this a bug, is the feature viable. needs author feedback Pending additional information from the author labels Mar 27, 2023
@ben-wright
Copy link
Author

Hi @barticus thank you for your response.

  1. Correct, I can see the users in FIrebase dashboard including users with hidden address ie @privaterelay.appleid.com
  2. Correct, users are being signed in directly from frontend library, no custom token is being used.

@ben-wright
Copy link
Author

Hi @barticus , on further inspection, I believe I am getting logs from mailchimp extension:

User does not have an email

When user is signing in with apple. But I'm not sure why the email is blank on signup?

I am using React Native and requesting email scope when user logs in:

const appleAuthRequestResponse = await appleAuth.performRequest({ requestedOperation: appleAuth.Operation.LOGIN, requestedScopes: [appleAuth.Scope.FULL_NAME, appleAuth.Scope.EMAIL], });

And also the email is also visible on Firebase dashboard in authentication section.

Any ideas why email would be blank on functions.handler.auth.user.onCreate trigger?

@barticus
Copy link
Collaborator

barticus commented May 9, 2023

Sorry for not getting back to you @ben-wright.

It sounds very strange! Can't find anyone else reporting this kind of issue. There are some issues around the full name not always being returned - might be worth a read to see if it impacts you, but I doubt its this same issue.

We've prioritised looking into this but it still might be a few weeks until we do. We will need to set up a basic app of some form to repro. If you need it faster than that, I've put details in the readme on how to set up a local firebase project with this extension enabled, you could pull the code and add some extra log statements to track down what is going on. I understand that is not ideal, but could get this done faster.

@ben-wright
Copy link
Author

Thanks for reply :) Good idea, I will setup locally to further test what is happening and try to replicate locally.

@barticus
Copy link
Collaborator

barticus commented Oct 4, 2023

Hi @ben-wright , did you get any further with this?

On our end, the person who was going to look at this was pulled on to another project, so it hasn't been assessed internally yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs author feedback Pending additional information from the author needs investigation Investigation needed to find more information, e.g. is this a bug, is the feature viable.
Projects
None yet
Development

No branches or pull requests

2 participants