-
Notifications
You must be signed in to change notification settings - Fork 0
Pull ozu data from inside worker #23
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
Changes from all commits
f28417e
f25d148
43bced9
9f6d4c5
da0c8c3
f6fb3b2
1ea357a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -12,7 +12,7 @@ | |||||
|
|
||||||
| class FetchDataFromOzu extends Command | ||||||
| { | ||||||
| protected $signature = 'ozu:fetch-ozu-data'; | ||||||
| protected $signature = 'ozu:fetch-ozu-data {--force : Do not ask for confirmation} {--withoutAssets : Do not download assets}'; | ||||||
|
||||||
|
|
||||||
| protected $aliases = ['ozu:fetch-data', 'ozu:pull']; | ||||||
|
|
||||||
|
|
@@ -26,14 +26,16 @@ class FetchDataFromOzu extends Command | |||||
|
|
||||||
| public function handle(Client $ozuClient): int | ||||||
| { | ||||||
|
|
||||||
| $this->warn('⚠️ This action will erase your local database and assets.'); | ||||||
|
|
||||||
| if (!confirm( | ||||||
| 'Are you sure you want to continue? This cannot be undone.', | ||||||
| default: false, | ||||||
| )) { | ||||||
| return self::SUCCESS; | ||||||
| $this->newLine(2); | ||||||
| $this->warn('⚠️ This action will erase your local database and assets.'); | ||||||
|
||||||
| $this->warn('⚠️ This action will erase your local database and assets.'); | |
| $this->warn('⚠️ This action will erase your local database and assets.'); |
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.
The option name
withoutAssetsshould use kebab-case following Laravel conventions. It should be--without-assetsinstead of--withoutAssets. This improves consistency with standard Laravel command options.