-
-
Notifications
You must be signed in to change notification settings - Fork 22
Add Laravel Prompts to command interface #19 #74
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: main
Are you sure you want to change the base?
Conversation
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.
Can you also take a look at the tests? They're failing now.
$this->info('Please wait while we scan your web page...'); | ||
$this->line(''); | ||
$url = $this->argument('url') ?? text( | ||
label: 'Pleaes enter the url', |
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.
There is a typo here
class SeoScanUrl extends Command | ||
{ | ||
public $signature = 'seo:scan-url {url} {--javascript}'; | ||
public $signature = 'seo:scan-url {url?} {--javascript}'; |
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.
What if I don't want to enter a url and want to scan the whole website?
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.
Ah never mind I see it's the Scan URL command 🙂
This pull request introduces several changes to enhance the functionality and user experience of the SEO scanning tool. The primary updates include the integration of the Laravel Prompts package, modifications to the
SeoScanUrl
command to improve user interaction, and updates to theSeo
class to utilize the new progress functionality.Integration and functionality improvements:
composer.json
: Addedlaravel/prompts
package to the dependencies.src/Commands/SeoScanUrl.php
: Updated the command to use Laravel Prompts for user input and progress indication. This includes changing theurl
argument to be optional and adding interactive prompts for the URL and JavaScript option.Class updates:
src/Seo.php
: Replaced the SymfonyProgressBar
with Laravel PromptsProgress
for better progress tracking. [1] [2]