search

zeyadsaleh / Laravel_BlogApp

0 stars 0 forks source link

update validation rules to match the lab requirements #2

Open mhmodmonim opened 4 years ago

mhmodmonim commented 4 years ago

you missed the uniqueness and user existence validations

https://github.com/zeyadsaleh/Laravel_BlogApp/blob/a83f4cd4e56020ce84954bf6840354884fbfdb5c/app/Http/Controllers/PostController.php#L86-L89

zeyadsaleh commented 4 years ago

ahaa , yeah i know and I've corrected it in my last commit, didn't it appear?

On Fri, 27 Mar 2020 at 22:47, Mahmoud Abdelmoneim notifications@github.com wrote:

you missed the uniqueness and user existence validations

https://github.com/zeyadsaleh/Laravel_BlogApp/blob/a83f4cd4e56020ce84954bf6840354884fbfdb5c/app/Http/Controllers/PostController.php#L86-L89

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/zeyadsaleh/Laravel_BlogApp/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK77M65A2TOHYHTG6U6MUXTRJUGH5ANCNFSM4LVI6A2Q .

zeyadsaleh commented 4 years ago

I'll push again, it seems like this part hasn't been pushed

On Fri, 27 Mar 2020 at 22:50, Zeyad Saleh zeyadmostafasaleh@gmail.com wrote:

ahaa , yeah i know and I've corrected it in my last commit, didn't it appear?

On Fri, 27 Mar 2020 at 22:47, Mahmoud Abdelmoneim < notifications@github.com> wrote:

you missed the uniqueness and user existence validations

https://github.com/zeyadsaleh/Laravel_BlogApp/blob/a83f4cd4e56020ce84954bf6840354884fbfdb5c/app/Http/Controllers/PostController.php#L86-L89

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/zeyadsaleh/Laravel_BlogApp/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK77M65A2TOHYHTG6U6MUXTRJUGH5ANCNFSM4LVI6A2Q .