amplify-js icon indicating copy to clipboard operation
amplify-js copied to clipboard

feat(auth): Adding `AnalyticsMetadata` to the cognito requests where applicable.

Open ivan-kiselev opened this issue 1 year ago • 1 comments

Description of changes

Propagating AnalyticsMetadata (doc) among with some cognito requests, where applicable, so cognito statistics are reflected in AWS Pinpoint analytics.

Issue #12877

Description of how you validated changes

SignUp request linked library produces, screenshot from browser:

Screenshot 2024-01-25 at 11 13 54

After confirming email, I can observe sign up event on Pinpoint Analytics dashboard

Checklist

  • [x] PR description included
  • [x] yarn test passes
  • [x] Tests are changed or added
  • [ ] Relevant documentation is changed or added (and PR referenced)

Checklist of requests to propagate metadata with

  • [x] ConfirmForgotPasswordRequest
  • [x] ConfirmSignUpRequest
  • [x] InitiateAuthRequest
  • [x] ResendConfirmationCodeRequest
  • [x] RespondToAuthChallengeRequest
  • [x] SignUpRequest

Testing in sample project

To test in sample project, one needs to yarn link both @aws-amplify/core and @aws-amplify/auth, because base Amplify config changed as well.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

ivan-kiselev avatar Jan 24 '24 12:01 ivan-kiselev

@elorzafe @nadetastic ping

ivan-kiselev avatar Feb 11 '24 10:02 ivan-kiselev

Closing it in an absence of any attention from maintainers

ivan-kiselev avatar Apr 12 '24 06:04 ivan-kiselev