UPDATE 10:42am UTC: Sorry I will stop reviewing the PRs, there's just too many of them, 50+ PRs in 5 hours! Will shoot a separate video in a few days, discussing the results, and for the correct answer check PRs from 1 to 4.
This is a demo-project which has intentional errors around Routing and Route Model Binding.
Your task is to fix those errors, by submitting a Pull Request.
- Clone the repository with git clone
- Copy .env.example file to .env and edit database credentials there
- Run composer install (if anyone got problems with composer on windows, try running it like this: composer install --ignore-platform-reqs)
- Run php artisan key:generate
- Run php artisan migrate --seed (it has some seeded data for your testing)
- That's it: launch the main URL.
You should see the list of transactions, something like this:
None of those three buttons View/Export/Duplicate will work. This is exactly the task for you.
I will be expecting a Pull Request to the main branch, containing all fixes with completely working project.
If you don't know how to contribute a PR, here's my video with instructions.
Important: I will NOT merge the Pull Request, only comment on it, whether it's correct or not.
At the time of writing, I'm not sure how many PRs I will be able to review, or maybe the community will help them. At some point, I will probably stop the challenge and stop accepting PRs, providing the correct answer, which may be actually chosen as one of the PRs.
If you have any questions, or suggestions for the future challenges, please open an Issue.
Good luck!
When you click View, the error should be something like this:
Target class [App\Http\Controllers\Transaction] does not exist.
Expected result, after the fix, is a page with task details:
You can perform this task only AFTER you finish the Task 1 above, otherwise you will get the same error as Task 1.
When you click Export, the error should be something like this:
Attempt to read property "name" on null (View: /resources/views/transactions/export.blade.php)
Expected result, after the fix, is a page for checking details for export:
Duplicate the task should happen with UUID as a URL parameter, it's one of the database fields: transactions.uuid.
When you click Duplicate, you may have two kinds of errors, randomly:
- Either is would show the incorrect transaction, so data from another transaction
- Or it would show the page "404 Not Found"
Expected result, after the fix, is a page for checking details for duplication:
