Skip to content

Conversation

@JonPurvis
Copy link
Collaborator

This is something I've wanted to add to this package for a while now and I finally dedicated some time to getting it done!

Historically, whenever you add a new connector, request, response etc.. you have to type the Integration it's for. This is fine, but if you make a typo or forget the name of the integration, you end up with an integration that either needs renaming or deleting and you have to start over.

This PR introduces the suggest Laravel Prompt so when you get asked for the Integration, you can now either select from your existing integrations or type in the name for a new one

Screenshot 2025-11-21 at 10 37 40

It's not immediately clear that you can type the name of a non-existing integration, so I've added some extra information in the hint of the prompt.

@JonPurvis JonPurvis assigned JonPurvis and unassigned JonPurvis Nov 21, 2025
@JonPurvis JonPurvis requested a review from Sammyjo20 November 21, 2025 10:42
@JonPurvis JonPurvis changed the title Select Existing Integration Suggest Existing Integrations Nov 21, 2025
Copy link
Member

@Sammyjo20 Sammyjo20 left a comment

Choose a reason for hiding this comment

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

Amazing work @JonPurvis - I'm happy to merge and release if you are!

@JonPurvis
Copy link
Collaborator Author

@Sammyjo20 Thanks for the speedy review! Yeah let's get it merged and released! 🚀 The less time I spent typing out integration names, the better 😆

@JonPurvis JonPurvis merged commit 85e4232 into saloonphp:v3 Nov 21, 2025
34 checks passed
@JonPurvis
Copy link
Collaborator Author

@Sammyjo20 I've merged it, I'll hold off on a release until you're ready!

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.

2 participants