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

Update integration page style #344

Merged
merged 2 commits into from
Feb 3, 2025
Merged

Conversation

satococoa
Copy link
Contributor

Summary

Enhanced the integration page UI and functionality.

GitHub is not connected.

スクリーンショット 2025-01-31 14 57 32

The "Connect" button redirects users to /settings/account.

GitHub is connected.

スクリーンショット 2025-01-31 14 57 54

Related Issue

Other Information

The GitHub connection-related code has become somewhat complex and needs refactoring in the near future.

- Redesigned GitHub Integration component layout
- Improved repository display with avatar and status
- Simplified installation and repository rendering
- Enhanced visual hierarchy and responsiveness
- Extracted presentation logic into separate components
- Improved type definitions for GitHub integration
- Simplified rendering of GitHub integration states
- Added more flexible handling of GitHub connection status
Copy link

vercel bot commented Jan 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
giselle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 31, 2025 6:01am

@satococoa satococoa self-assigned this Jan 31, 2025
@satococoa satococoa marked this pull request as ready for review January 31, 2025 06:02
Copy link
Member

@shige shige left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@satococoa satococoa merged commit f236b0c into main Feb 3, 2025
9 checks passed
@satococoa satococoa deleted the update-integration-page-style branch February 3, 2025 00:03
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