React Native – "identifyUser" function does not work on launch #13504
Comments
|
As an update, if I comment out / remove this if statement
export const identifyUser: IdentifyUser = async ({
userId,
userProfile,
options,
}) => {
assertIsInitialized();
const { credentials, identityId } = await resolveCredentials();
const { appId, region } = resolveConfig();
const { address, optOut, userAttributes } = options ?? {};
if (!(await getEndpointId(appId, 'PushNotification'))) {
// if there is no cached endpoint id, wait for successful endpoint creation before continuing
await getInflightDeviceRegistration();
}
await updateEndpoint({
address,
channelType: getChannelType(),
optOut,
appId,
category: 'PushNotification',
credentials,
identityId,
region,
userAttributes,
userId,
userProfile,
userAgentValue: getPushNotificationUserAgentString(
PushNotificationAction.IdentifyUser,
),
});
};So it seems like the |
chrisbonifacio
added
React Native
|
As an update, I tried logging from the import { PushNotificationError } from '../../../errors';
import {
InflightDeviceRegistration,
InflightDeviceRegistrationResolver,
} from '../types';
const inflightDeviceRegistrationResolver: InflightDeviceRegistrationResolver =
{};
let inflightDeviceRegistration: InflightDeviceRegistration = new Promise<void>(
(resolve, reject) => {
inflightDeviceRegistrationResolver.resolve = resolve;
inflightDeviceRegistrationResolver.reject = reject;
},
);
export const getInflightDeviceRegistration = () => inflightDeviceRegistration;
export const resolveInflightDeviceRegistration = () => {
inflightDeviceRegistrationResolver.resolve?.();
// release promise from memory
inflightDeviceRegistration = undefined;
};
export const rejectInflightDeviceRegistration = (underlyingError: unknown) => {
inflightDeviceRegistrationResolver.reject?.(
new PushNotificationError({
name: 'DeviceRegistrationFailed',
message: 'Failed to register device for push notifications.',
underlyingError,
}),
);
// release promise from memory
inflightDeviceRegistration = undefined;
}; |
|
I investigated some more and figured what goes wrong. To me, it looks like a bug. When calling The problem is that the register device function fails until a user is authenticated. This is fine and makes sense. However, the issue is that the exception is not passed on to This incorrect behavior because the device was never successfully registered, but the code now thinks it's initialized. Hence, whenever calling In my estimation, this should not happen. We should set export const initializePushNotifications = (): void => {
if (isInitialized()) {
logger.info('Push notifications have already been enabled');
console.log('Push notifications already initialized')
return;
}
addNativeListeners();
addAnalyticsListeners();
initialize();
};Source: |
|
It seems like the ticket is related to my other open issue. We should continue the conversation in this thread for simplicity. |
cwomack
added
question
|
@julian-dotcom per your suggestions above, closing this in favor of your other issue. |
|
Sorry for the confusion/misunderstanding, but I think we should continue in this thread right here, and close the other one. |
Also experienced this issue and after commenting out the |
Before opening, please confirm:
JavaScript Framework
React Native
Amplify APIs
Analytics, Push Notifications
Amplify Version
v6
Amplify Categories
notifications
Backend
None
Environment information
Describe the bug
identifyUserfromaws-amplify/push-notificationsnever successfully saves the endpoint with the address on first launch.It never registers the endpoint, until I close the app and restart it with the user already authenticated. On second launch, it succesfully submits the endpoint on Android, but never does on iOS.
Expected behavior
I sign in and upon sign in, the
identifyUserregisters the endpoint on first call, allowing me to receive notifications.Reproduction steps
unexpected behavior arises when I call
identifyUserCode Snippet
My index.js
How I register the device token:
identifyUsercall the function in this file/node_modules/@aws-amplify/notifications/src/pushNotifications/providers/pinpoint/apis/identifyUser.native.tswhere unfortunately,await getEndpointId(appId, 'Analytics')returnsundefinedLog output
aws-exports.js
Manual configuration
No response
Additional configuration
No response
Mobile Device
No response
Mobile Operating System
No response
Mobile Browser
No response
Mobile Browser Version
No response
Additional information and screenshots
No response
The text was updated successfully, but these errors were encountered: