Skip to content

docs(identity): Document behavior of linking social sign-ins #235 #682

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

Closed
wants to merge 4 commits into from

Conversation

Ayushmore1214
Copy link
Contributor

This PR adds documentation under identity/ explaining the behavior of linking multiple social sign-in accounts in Layer5 Cloud.

Notes for Reviewers

This PR fixes #235

Signed commits

  • Yes, I signed my commits.

…#235

This PR adds documentation under `identity/` explaining the behavior of linking multiple social sign-in accounts in Layer5 Cloud.


Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Copy link

netlify bot commented Jul 15, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 12aaa78
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/6878f5d93d34330008b30979
😎 Deploy Preview https://deploy-preview-682--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@zihanKuang zihanKuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Ayushmore1214, you shouldn't create a new page for this. You can first check if the content from the issue has been written on this page(https://docs.layer5.io/cloud/identity/users/#account-linking). If not, please add it. If it has been written, we can close this issue. 👍

@miacycle
Copy link
Contributor

Hi @Ayushmore1214, you shouldn't create a new page for this. You can first check if the content from the issue has been written on this page(https://docs.layer5.io/cloud/identity/users/#account-linking). If not, please add it. If it has been written, we can close this issue. 👍

Good advice. Thank you for offering guidance here, @zihanKuang.

@Ayushmore1214, have you reflected on the existing content?

Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
@Ayushmore1214
Copy link
Contributor Author

Yes @miacycle I reviewed the existing content , I noticed that the specific behaviors mentioned in issue #235 were missing — particularly , So i added them plz take a lookk , Thank You

Copy link
Contributor

@zihanKuang zihanKuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Ayushmore1214, I have a couple of questions:

  1. Have you personally tested the behavior described in the issue? Could you clarify what "primary account" refers to?
  2. Why is this content placed under "Account Deletion" instead of "Account Linking"?

@Ayushmore1214
Copy link
Contributor Author

Yes, I did test this behavior personally to make sure the documentation reflects how it actually works. Good catch. By "primary account," I meant the first social account a user signs up with. Ypu are right the location is messed up I'll fix it

Signed-off-by: Ayush More <ayushmore42595@gmail.com>
@vr-varad
Copy link
Contributor

Thanks for the effort! The changes are quite different from what's needed and would require several iterations. Let's close this PR for now.

@vr-varad vr-varad closed this Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cloud] Behavior of linking of social sign-ins
4 participants