Size: a a a

2019 September 06

В

Вячеслав in Laravel UA
Да, там нужно было протестировать в двух вариантах, с типом и без
источник

В

Вячеслав in Laravel UA
Описание не говорит о пользе
источник

В

Вячеслав in Laravel UA
Ты пишешь, что хранить namespace в БД, плохо, а почему, не понятно. Что конкретно станет лучше если хранить там кастомную строку.
источник

O

Ostap 🇺🇦 in Laravel UA
спасибо за совет
источник

O

Ostap 🇺🇦 in Laravel UA
Вячеслав
Ты пишешь, что хранить namespace в БД, плохо, а почему, не понятно. Что конкретно станет лучше если хранить там кастомную строку.
размер столбца, економия места?)
источник

O

Ostap 🇺🇦 in Laravel UA
ааа, не совсем
источник

O

Ostap 🇺🇦 in Laravel UA
namespace может изменится, а тип текстом - нет
источник

O

Ostap 🇺🇦 in Laravel UA
пойду допишу
источник

O

Ostap 🇺🇦 in Laravel UA
не знаю, можно ли добавить еще коммиты в существующий пулл
источник

В

Вячеслав in Laravel UA
Это уже лучше
источник

В

Вячеслав in Laravel UA
Можно, просто пуш новый коммит
источник

O

Ostap 🇺🇦 in Laravel UA
ого, спасибо!)
источник

В

Вячеслав in Laravel UA
И можешь описать, что мотивация такая же как и в случае с типами полиморфных связей:
https://laravel.com/docs/6.0/eloquent-relationships#custom-polymorphic-types
источник

В

Вячеслав in Laravel UA
Скорее всего реализацию нужно сделать такуюже
источник

В

Вячеслав in Laravel UA
Так шансов уже будет побольше 🙂
источник

O

Ostap 🇺🇦 in Laravel UA
хорошо, подумаю
источник

O

Ostap 🇺🇦 in Laravel UA
тоесть прямо связать с MorphMap?
источник

O

Ostap 🇺🇦 in Laravel UA
ооо, тоесть добавить ее поддержку
источник

O

Ostap 🇺🇦 in Laravel UA
идея для следующего пула 🙂
источник

O

Ostap 🇺🇦 in Laravel UA
спасибо
источник