Skip to content

Add sign-in button to initial screen#1926

Closed
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin/1740713375-add-sign-in-button
Closed

Add sign-in button to initial screen#1926
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin/1740713375-add-sign-in-button

Conversation

@devin-ai-integration
Copy link
Copy Markdown

Added a sign-in button to the initial screen (DeviceSelectionPage) that follows the existing design patterns and allows users to sign in.

Link to Devin run: https://app.devin.ai/sessions/c0ed8b2cf0b94c2490dac251c24a96be

Co-Authored-By: Nik Shevchenko <kodjima33@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@kodjima33
Copy link
Copy Markdown
Collaborator

just testing Devin

@beastoin
Copy link
Copy Markdown
Collaborator

ohhh bro

Consumer<AuthenticationProvider>(
builder: (context, authProvider, child) {
return ElevatedButton(
onPressed: () {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Bro Devin this will sign the user in using Google. We have a screen named AuthWrapper and when a user presses on Sign In they should be redirected to that screen because that screen already handles both google and apple auth. And don't use an elevated button, rather use something like text button because this is a secondary button

@mdmohsin7
Copy link
Copy Markdown
Member

Fix it pls Devin 😁

…pper

Co-Authored-By: Nik Shevchenko <kodjima33@gmail.com>
@beastoin
Copy link
Copy Markdown
Collaborator

beastoin commented Mar 3, 2025

fixed #1932

@beastoin beastoin closed this Mar 3, 2025
@beastoin
Copy link
Copy Markdown
Collaborator

beastoin commented Mar 3, 2025

* not by devin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants