пока идея с виспером мне больше нравится. нет по пути мешающих сущностей

а ларой можно по апи принять местоположение плеера и обработку раз в несколько секунд делать

В общем все получилось) Через виспер. Только у pusher-js оно через channel.trigger клиентского события

пинг на вайфае 30-60 мс

это сумма пинг-понг событий от одного конца до второго

Всем привет, решил я тут чуть заняться написанием пакетов, был один который я хотел недавно юзануть, он год без поддержки, чуть нужно было поправить, я поправив его написал еще один который не мог найти на просторах интернета для laravel. Хотелось бы получить что-то code review, ну или просто хотя бы взгляд со стороны 😉

https://github.com/maksa988/laravel-unitpay
https://github.com/maksa988/laravel-freekassa

В будущем планирую еще тесты добавить, как только разберусь с ними под лару

Даже так) благодарю

Конечно, мнение приветствуется, учимся вместе! 😉

Я начну, смотрю laravel-unitpay:
1. Нет тестов. Что не есть хорошо и понижает уровень доверия к решению. Если на коммерческих проектах тесты пишутся далеко не всегда, то в Open Source - это маст хев.
2. Best Practice по разработке пакетов (особенно в Laravel сообществе), предусматривают подключение Travis CI, Style CI. Для проверки логики и стилей.
3. Не понятно для чего вот это в автозагрузчике - https://github.com/maksa988/laravel-unitpay/blob/master/composer.json#L21 т.к. у тебя уже загружается папка src
4. Установлена жесткая зависимость от всего Laravel Framework. Лучше устанавливать зависимости от конкретных illuminate компонентов. Таким образом это дает возможность использовать пакет не только с Laravel Framework.
5. Ключи в конфиг файле имеют разный формат - есть searchOrder в camelCase, есть pay_url в snake_case. Все ключи конфига должны быть наименованы в snake_case
6. Для кастомизации сообщений лучше использовать не конфиг, а lang файлы. Таким образом ты обеспечиваешь и редактируемость и мультиязычность.
7. Не совсем понятно, почему часть логики вынесена именно в Трейты.

Если в общем, хорошо, что решил попробовать себя в OpenSource. Очень полезное занятие, может помочь тебе подтянуть различные навыки. Такие работы очень полезны в плане достежения небольшого, но видимого результата.

Оу, спасибо. По поводу тестов я их просто как-раз изучаю, поэтому решил не писать на быструю руку.
По остальным моментам разберусь и поправлю. Некоторые моменты просто брал из "примеров", поэтому может быть и отличия в форматах ключей тех же и что-то еще.

Всім привіт у мене знову питання по join і пагінації
і так є таблиця з маршрутом яку ми відсортовуємо по даті останньої точки, це відбувається так
select
 `waypoints`.`date`,
 `orders`.*
from
 `orders`
 inner join `waypoints` on `orders`.`id` = `waypoints`.`order_id`
 and waypoints.priority = (
   select
     waypoints.priority
   from
     waypoints
   where
     orders.id = waypoints.order_id
   order by
     waypoints.priority desc
   limit
     1
 )
order by
 `waypoints`.`date` desc
limit
 25 offset 0
Тобто я роблю підзапит де визначаю найбільший пріорітет і вибираю точку з найбільшим пріорітетом, сам запит відпрацьовує швидко, але з пагінацією запит стає таким
select
 count(*) as aggregate
from
 `orders`
 inner join `waypoints` on `orders`.`id` = `waypoints`.`order_id`
 and waypoints.priority = (
   select
     waypoints.priority
   from
     waypoints
   where
     orders.id = waypoints.order_id
   order by
     waypoints.priority desc
   limit
     1
 )
Відповідно для кожного вантажа відпрацьовує підзапит, питання в тому чи можна обійтись без підзапита, наприклад в запиті на пріорітет ставити 1 для count  і підзапит для виборки? або чи можна взагалі без підзапиту обійтись (це було б ідеально)? які думки у вас?

продублював на тостері, https://toster.ru/q/598349

Привіт люди!

Можете покритикувати код?

Дякую

https://paste.laravel.io/2f09341e-38c0-4349-9b6d-109985eb0704

- в контроллерах не должно быть логики. Всю логику перенести в сервисы, и вызывать их из контроллера
- штуки в стиле auth()->check() старайся не юзать, это неявные зависимости. Лучше внедрить через DI Request в экшн и использовать $request->user() для проверки.
- почему-то используешь работу с БД вместо работы с Eloquent(ORM). Когда достаешь $lessons_count - его точно можно достать через Eloquent и связи. Подозревая, что и $course тоже, но не знаю, разобраться, какая логика там нужна достаточно сложно.

region.cities👍🏻

ничего не знаю про сервисы, буду гуглить

спасибо!

phptherightway чтоль читать...

Если несложно, скиньте курс/книгу/сайт где можно научится нормально делать, так-же, как люди делают сложные приложения, буду очень благодарен...

а именно как правильно внедрять сложную бд в ларавел