Closed nimrodkor closed 3 years ago
Could you fix the merge conflicts?
@amazon-meaisiah Done!
Hey again @amazon-meaisiah !
I tested it out on our environment - and for some reason, the URL for the existing cognito user pool is broken in the UI. When I try to sign in, it sends me to https://developers.test.com/<COGNITO_URL>
Instead of https://<COGNITO_URL>
. Tried to fix it but had a hard time. Any possible help here?
@nimrodkor Not sure personally based on a cursory look at the code, but I'll see if I can work with other team members to figure out what's going on or if I just missed something.
@amazon-meaisiah Removed the temp files, sorry about that!
@nimrodkor - Looks like this change was merged in a while ago but it is not in the latest any more. Any ideas on what happened to it? Were there any issues with this? thanks
Description of changes: Added two parameters to allow usage of existing User Pool instead of creating a new one.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.