-
-
Notifications
You must be signed in to change notification settings - Fork 254
Add facebook social sign-in to bit Boilerplate (#11038) #11041
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
Conversation
WalkthroughThe changes introduce Facebook social sign-in support to the Bit Boilerplate. This includes UI components for a Facebook login button and icon, backend configuration for Facebook authentication, necessary package references, and updated localization and configuration files to support the new provider. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant ClientApp
participant ServerApi
participant Facebook
User->>ClientApp: Clicks "Sign in with Facebook"
ClientApp->>ServerApi: Initiates Facebook OAuth flow
ServerApi->>Facebook: Redirects for authentication
Facebook->>User: Prompts for login/consent
Facebook->>ServerApi: Returns authentication response
ServerApi->>ClientApp: Issues authentication token/session
ClientApp->>User: User is signed in
Assessment against linked issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (11)
💤 Files with no reviewable changes (1)
🧰 Additional context used🪛 Biome (1.9.4)src/Templates/Boilerplate/Bit.Boilerplate/src/Server/Boilerplate.Server.Api/appsettings.json[error] 5-5: End of file expected Use an array for a sequence of values: (parse) [error] 5-5: End of file expected Use an array for a sequence of values: (parse) [error] 5-5: End of file expected Use an array for a sequence of values: (parse) [error] 5-5: End of file expected Use an array for a sequence of values: (parse) ⏰ Context from checks skipped due to timeout of 90000ms (1)
🔇 Additional comments (11)
✨ Finishing Touches
🧪 Generate Unit Tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for Facebook social sign-in to the Bit Boilerplate by introducing resource strings, configuration sections, package references, backend auth setup, and UI components.
- Added new resource entries and translations for Facebook sign-in button text
- Updated
appsettings.jsonfiles and authentication setup to include Facebook - Introduced UI click handler, button markup, and an SVG icon for Facebook
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/.../AppStrings.resx | Add FacebookSignInButtonText resource |
| src/.../AppStrings.fa.resx | Add Farsi translation for Facebook sign-in button |
| src/Server/Boilerplate.Server.Web/appsettings.json | Add Facebook config section without comment |
| src/Server/Boilerplate.Server.Api/appsettings.json | Add Facebook config section with docs comment |
| src/Server/Boilerplate.Server.Api/Program.Services.cs | Register Facebook authentication when AppId is set |
| src/Server/Boilerplate.Server.Api/Boilerplate.Server.Api.csproj | Add Microsoft.AspNetCore.Authentication.Facebook |
| src/Directory.Packages.props | Pin Facebook auth package version 9.0.6 |
| src/Client/.../SocialRow.razor.cs | Add HandleFacebook() click handler |
| src/Client/.../SocialRow.razor | Render Facebook button when supported |
| src/Client/.../FacebookIcon.razor | New FacebookIcon SVG component |
Comments suppressed due to low confidence (1)
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Identity/Components/SocialRow.razor:50
- No tests were added to verify that
Facebookappears insupportedProvidersand renders the button; consider adding a UI or integration test for the Facebook flow.
@if (supportedProviders.Contains("Facebook"))
closes #11038
Summary by CodeRabbit
New Features
Style