-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Feat/GitHub Make GitHub repository dropdown searchable in new app flow #7064
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
base: next
Are you sure you want to change the base?
Feat/GitHub Make GitHub repository dropdown searchable in new app flow #7064
Conversation
v4.0.0-beta.438
- Replace x-forms.select with x-forms.datalist for repository picker - Enable type-to-filter UX with Alpine.js, keeping Livewire binding - Update in resources/views/livewire/project/new/github-private-repository.blade.php
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.
Please change the target branch to NEXT instead of v4.x and did you manually test your changes?
|
Alright. I’ll change the target branch to next. I also tested my changes.
…On Fri, Oct 31, 2025 at 5:49 AM ShadowArcanist ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Please change the target branch to NEXT instead of v4.x and did you
manually test your changes?
—
Reply to this email directly, view it on GitHub
<#7064 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACFRIUILJ22PW6Q555KPHMD32NLFFAVCNFSM6AAAAACKYPGNLOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTIMBUGEZTCNBZHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
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.
@ShadowArcanist I've updated the target branch from v4.x to next as requested.
The changes remain the same:
- Replaced
x-forms.selectwithx-forms.datalistfor searchable repository selection - All functionality preserved with improved UX
Ready for review!
|
@CodeRabbit review |
✅ Actions performedReview triggered.
|
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (2)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Summary by CodeRabbit
WalkthroughA Blade template refactors the GitHub repository selector from a Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant Datalist as Datalist Component
participant Livewire as Livewire (wire:model.live)
User->>Datalist: Type to search repositories
Datalist->>Livewire: Emit live input changes
Livewire->>Datalist: Update available options
Datalist->>User: Display filtered repositories
User->>Datalist: Select repository
Datalist->>Livewire: Commit selection
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
I'll be back... to deploy these configs on a proper self-hosted server, not some serverless nonsense that VC firms dreamed up to extract your cash. At least you can SSH into a real machine and grab a gluten-free taco while your infrastructure actually runs instead of spinning up cold lambdas. 🌮 Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
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. Comment |
Thanks! I just tested it and works fine for me! You have changed services templates on your commit but it's not mentioned on your commit message or in PR description, please revert it (Keep one change per PR, if you like to make changes to service then do a separate PR for it) |
- Revert changes to templates/service-templates.json and templates/service-templates-latest.json - These changes were accidentally included in the repository search dropdown PR - Service template updates should be in a separate PR
|
@ShadowArcanist I've updated reverted the changes committed in error to templates/service-templates.json and templates/service-templates-latest.json |

Changes
Issues
Testing