#Unable to set expo credentials.

1 messages · Page 1 of 1 (latest)

lunar laurel
#

Hey guys, I am using the hosted service, trying to send an expo notification. I have integrated expo and added a subscriber, both in the dev environment. When I try to add the expo token using the setCredentials function, I get an HTTP 400 response. I have added expo in the dev environment and the subscriber also belongs to the dev environment.

I have attached pictures of the code I am running and the response I receive.

split mountain
#

Are you sure that ExponentPushToken[] should be in the device token?

#

and what is the response on setting credentials on subscriber?

lunar laurel
#

Hey, yes ExponentPushToken is a part of the token.

And the setCredentials function is what gives me the above response.

jolly trailBOT
#

@lunar laurel, you just advanced to level 1!

lunar laurel
#

Is there a solution for this?

#

Maybe some way that I can test if expo has actually integrated with novu?

split mountain
#

@lunar laurel have you checked so the expo integration is activated in integration store?

lunar laurel
#

Yup, it says active

split mountain
#

and you are using the api key for the correct environment?

lunar laurel
#

Yup

split mountain
#

okay could you create a bug for this on github?

lunar laurel
#

Sure

split mountain
lunar laurel
#

Hey, I got the issue, the PushProviderEnum does not have an expo option, I was passing a string direvtly, but it said 'EXPO' instead of 'expo' in lower case.

split mountain
#

ah did that solve it?

lunar laurel
#

Yup, wokrs now

split mountain
#

nice good find

lunar laurel
#

👍

#

Should I create an isse?

#

*issue?

split mountain
#

no I will look into how it should be 🙂

lunar laurel
#

Cool

split mountain
#

All of our provider id are in lower case so it seems to be okay but maybe we should fix so it is lowercased when we receive them in the set credentials endpoint...

lunar laurel
#

Sure, but the only reason I chose to enter a string directly is that there was no expo option in the PushProviderIdEnum enum. So I think adding the expo option there should be enough

split mountain
#

what version of Novu are you using?

lunar laurel
#

The latest

#

I just pulled to ensure I was not using an older versions

#

*version

split mountain
#

great will look into why it is not there 🙂