Алексей Уколов

чтобы не могли взять например заранее сгенерированный пароль и другую важную информацию

Важная информация не должна покидать сервер, если требуется, чтобы её не знал пользователь.

По итогам ревью код может измениться кардинально, что может повлечь за собой новые баги и исправление существующих.
По итогам тестирования код редко значительно изменяется - обычно правятся какие-то локальные баги.
Полноценное тестирование - это обычно более ресурсоёмкий процесс, чем ревью.

Исходя из этого - сначала ревью.

const newElemProps = {
  [el]: !old[el]
};

"Работать как с обычным массивом" нужно внутри колбэка - асинхронный код синхронным не сделать. Есть промисы и async/await, но они принципиально сути не меняют.

$user2->roles()->attach($manager);
$user2->permissions()->attach($mangeUsers);

Пользователя нужно сохранить сначала, как в первом случае.

Как реализовать вывод картинок и удаление при редактировании? Что почитать для этого?

Ответ на первый вопрос:

<img src="{{$image->path }} " alt="{{$image->title}}">

+ https://laravel.com/docs/7.x/filesystem#file-urls

Ответ на второй: https://laravel.com/docs/7.x/filesystem#deleting-files

Ответ на третий: https://laravel.com/docs/7.x

Где у вас резолв промиса-то, откуда данным взяться?

Потому что чтобы что-то вернуть, нужно использовать return.

Если бы вбили свой вопрос в поисковик, то увидели бы древнее (и единственно верное в данных условиях) решение - плейсхолдеров должно быть столько же, сколько значений и биндить нужно каждое значение отдельно, а не конкатенированную строку. Ну и в данном случае проще использовать не именованные параметры, а индексные, тогда можно сделать так:

$placeholders = substr(str_repeat('?,', count($ids)), 0, -1);

EGJVG-NTHQD-BJSUT-NZOJM-YWOLH в base64.

Удалить то, что, видимо, случайно закопипастилось в конце строки. Даже здесь вам подсветка синтаксиса указывает где проблема.

Адблок вырезает.

нынче не комильфо использовать php внутри html кода, то есть делать какие либо вставки php кода

Думаю, речь шла про то, что нужно использовать шаблонизаторы типа Twig, а не лапшу в стиле шаблонов Битрикса.

Прочитайте «Код» Петцольда, его как раз недавно переиздали.

user@513660de4e59:/var/www$ art tinker
Psy Shell v0.10.4 (PHP 7.4.8 — cli) by Justin Hileman
>>> public_path().'uploads/product_images';
=> "/var/www/publicuploads/product_images"
>>> public_path('uploads/product_images');
=> "/var/www/public/uploads/product_images"

Потому что меняется сам объект state и исходя из этого props компонента - mapStateToProps же каждый раз новый объект возвращает.