-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Hard to install, many errors #142
Comments
🚀 Here's the PR! #144💎 Sweep Pro: You have unlimited Sweep issues
Actions
Step 1: 🔎 Searching(Click to expand) Here are the code search results. I'm now analyzing these search results to write the PR.ecommerce-laravel/database/migrations/0001_01_01_000000_create_users_table.php Lines 1 to 50 in e09131f
ecommerce-laravel/database/migrations/2023_09_26_113743_create_customers_table.php Lines 1 to 35 in e09131f
ecommerce-laravel/database/migrations/2023_09_26_113752_create_orders_table.php Lines 1 to 33 in e09131f
ecommerce-laravel/database/migrations/2023_09_28_132432_create_order_items_table.php Lines 1 to 32 in e09131f
ecommerce-laravel/database/migrations/2023_09_30_151612_create_invoices_table.php Lines 1 to 32 in e09131f
ecommerce-laravel/database/migrations/2023_10_01_000000_create_inventory_logs_table.php Lines 1 to 25 in e09131f
ecommerce-laravel/app/Models/PaymentMethod.php Lines 1 to 16 in e09131f
ecommerce-laravel/app/Models/Subscription.php Lines 1 to 38 in e09131f
ecommerce-laravel/database/seeders/DatabaseSeeder.php Lines 1 to 21 in e09131f
ecommerce-laravel/app/Http/Controllers/Controller.php Lines 1 to 11 in e09131f
ecommerce-laravel/database/migrations/2023_04_01_000000_create_site_settings_table.php Lines 1 to 29 in e09131f
ecommerce-laravel/database/migrations/2023_04_03_000000_create_reviews_table.php Lines 1 to 32 in e09131f
ecommerce-laravel/database/migrations/0001_01_01_000002_create_connected_accounts_table.php Lines 1 to 41 in e09131f
ecommerce-laravel/app/Models/Customer.php Lines 1 to 43 in e09131f
ecommerce-laravel/app/Models/Order.php Lines 1 to 25 in e09131f
Step 2: ⌨️ CodingWe decided to make the following renames to help you solve the GitHub issue: I'm going to follow the following steps to help you solve the GitHub issue:
Here are the changes we decided to make. I'm done making edits and now I'm just validating the changes using a linter to catch any mistakes like syntax errors or undefined variables:
|
You removed the foreign keys reference and cascade, the correct fix for database/migrations/2023_09_30_151612_create_invoices_table.php is probably this: This issue doesn't seem to be resolved: |
We are looking for pull requests. Thank you for reporting this issues We are slowly working through all projects failing, plus testing. |
I've fixed migrations and install.yml now passes @geimsdin |
Please verify my sweep and manual changes. Install now seems to work. |
Details
1 - Migrations order is wrong, dependant tables are created before their dependency causing an error
2 - Migrations content has duplicated foreign keys
3 - Text encoding is wrong and many php files have the php opening tag converted in html making the file unreadable
4 - Some public functions are outside the class causing a fatal error
The text was updated successfully, but these errors were encountered: