ergodnc (Ergonomic Desk & Coffee) is an open source Laravel project that's being built live on the official Laravel YouTube Channel

In the OfficeImageController/store function, it returns an error when uploading the image via postman "Call to a member function storePublicly() on null" it cannot read the image file posted!! But the test passed successfully!!

When we created the office migration, the default approval_status is pending (1).

Why do we also need to do the following in the controller when creating an office?

$attributes['approval_status'] = Office::APPROVAL_PENDING;

NOTE: If we remove that line in the controller, the JsonResource response when creating an office does NOT contain the default pending status from the database. This will make the following assertion to fail:

->assertJsonPath('data.approval_status', Office::APPROVAL_PENDING)

I'm getting a weird error whenever I try to execute the tests. It seems like the first part of the app URL is missing.

Problems comes from OfficeImageControllerTest

And here's the file OfficeImageControllerTest

Can anyone help ?

What if the reservation date range is wider than the search date range !? The search date range will be included in the reservation date range but the reservation will not be shown in the result. But the office in fact will be reserved and occupied in this date search range. EX: Search: "2021-02-01" to "2021-04-01" Reservation: "2021-01-01" end "2021-05-01" "2021-01-01" not between "2021-02-01" & "2021-04-01" "2021-05-01" not between "2021-02-01" & "2021-04-01" but ["2021-02-01" & "2021-04-01"] is inside ["2021-01-01" & "2021-05-01"]

Hi @themsaid ,

I have cloned this repository with all settings as it is and for frontend, I am using your "nuxt-ergodnc" repository.

I have install front end using 'npm install' and backend using 'composer install && php artisan migrate && php artisan db:seed'

when, I try to login from frontend, It throws error '419 unknown status'.

I just noticed that it's not working in any of Laravel 8.x version. I tried it with Laravel 9 and it's working with Laravel 9 .

I think, there are some "cors " issues with "fruitcake/cors" package in laravel 8, Laravel 9 does not use this package. is there a way to make it work with laravel 8?

Thanks

• In the first, we used user_id for Hosts and visitor_id for visitors
• Is it better to use policies for user reservation authorization (not allowed to reserve its own offices)?
• Scope activeBetween should be named activeBetweenDates for consistency because for name convenience "activeBetween" can be either dates or numbers or ... so it's more efficient to rename it as "activeBetweenDates"

• What if I need an urgent reservation on the same date? the whole app will not be available for me
• If there is an available office on the today date it cannot be booked for today! so all the app will be again not available for all today booking. so the validation should be after_or_equal start-date for the end_date and we should remove the numberOfDays condition < 2

• Missing the validate() function on validator() helper
• Changing the key messages related to the date to "date_range" instead of "office_id"
• The diffDays return a floor result so to get the real number of days you should use (int)ceil() and floatDiffInRealDays: _$numberOfDays = (int)ceil(Carbon::parse(request('end_date'))->endOfDay()->floatDiffInRealDays( Carbon::parse(request('start_date'))->startOfDay() )); so now the difference between addDays(1) and addDays(40) will be 40 as you are getting the startDay of the first one and the endDate of the second one. Note: (Change in the testing 41 to 40)

**Error:**
- the calculation between today and tomorrow is 1.99999 which is 1 for floor and 2 for ceil functions

- If you use only addition by one the calculation will be wrong

• If you want to keep the logic of a 1-day reservation not allowed remove the condition because it already exists inside the validator when using the after:start_date for the end_date

Hi Mohamed

I think you forgot to call validate method on the validator helper in "UserReservationController" when you want to create a reservation

Hey Said, I am running with some issue within validators which validate the reservation for an office which should always be greater than 1 day. Lets say i make a reservation for an office from "2021-10-07" to "2021-10-08" on 2021-10-06 since the difference between these two dates is 1 day($numberOfDays = 1 ) and as user are allowed to stay till the last day which is 2021-10-08, technically that makes it reservation for 2 days( $numberOfDays + 1) and following validator will always pass if($numberOfDays < 2){ throw ValidationException::withMessages([ 'start_date' => 'You cannot make reservation for only one day' ]); } and user will be allowed to make the reservations which should not be allowed according to the validation rule.

Please let me know if i am mistaken.

Your code is using 'public' disk everywhere. I know you said that you would move this to s3 maybe in production, but this means changing all public disk use to s3 but you may have other uses for the public disk.

I would prefer to create a new disk just for images which initially maps to local but can change to map to an S3 bucket later.

I find it useful to store the disk name in the images table. This allows images to migrated from one disk to another at some point in the future.

API should return the full URI of the image so that the consumer does not need to know where the images are being stored and can just use the URI directly.

If email verification is switched on a user will need to be redirected back to the front end login page not the API.

This PR adds the redirect back to the front end.

Hi, I've added some pipelines that made the code much simple and also filtration classes that can be reusable. ref: https://laravel.com/api/8.x/Illuminate/Pipeline/Pipeline.html hope it can be useful.

all test passes!

finally I wanted to thank you for this amazing series.

best regards

in OfficeImageControllerTest after changing the imageResource to return the full path the Storage::assertExists($response->json('data.path')); return an error