Skip to content
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

feat: show social sign in on credentials select #343

Closed
wants to merge 2 commits into from

Conversation

aeneasr
Copy link
Member

@aeneasr aeneasr commented Jan 28, 2025

Adds a story book component for identifier first credentials select.

Closes #342

Screenshot 2025-01-28 at 10 51 45 Screenshot 2025-01-28 at 10 51 34 Screenshot 2025-01-28 at 10 51 30

Adds a story book component for identifier first credentials select.

Closes #342
@aeneasr aeneasr requested a review from jonas-jonas January 29, 2025 09:34
Copy link

codecov bot commented Jan 29, 2025

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 42.13%. Comparing base (f3fad4d) to head (67ca9af).
Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #343      +/-   ##
==========================================
- Coverage   42.43%   42.13%   -0.30%     
==========================================
  Files         136      138       +2     
  Lines        2008     2110     +102     
  Branches      288      313      +25     
==========================================
+ Hits          852      889      +37     
- Misses       1149     1209      +60     
- Partials        7       12       +5     
Components Coverage Δ
@ory/elements-react 36.92% <29.83%> (+0.13%) ⬆️
@ory/nextjs 64.33% <74.24%> (-1.65%) ⬇️

@aeneasr aeneasr enabled auto-merge (squash) January 29, 2025 12:28
Copy link
Member

@jonas-jonas jonas-jonas left a comment

Choose a reason for hiding this comment

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

If the user has multiple methods available, the second step looks like this:
image

The OIDC buttons on the first no longer work for me.

Screen.Recording.2025-01-29.at.4.37.44.PM.mov

return !exclude.includes(node.group)
})

console.log(otherNodes)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
console.log(otherNodes)

? listOnly(Object.keys(providers).filter((key) => providers[key]))
: LoginFlowFromJSON(oidcNodes)

console.log({ flow, oidcNodes })
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
console.log({ flow, oidcNodes })

@jonas-jonas
Copy link
Member

I don't think this is the right approach. #358

auto-merge was automatically disabled February 17, 2025 13:48

Pull request was closed

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.

Social sign in options not shown on identifier first credential chooser
2 participants