AЬ
С андерскором питонячья фишка больше
Size: a a a
2020 September 30
N
А, понял
N
Спасибо
I
Null
Всё понял кроме
is_logged нейминг нарушает конвенцииcamelCase vs snake_case
AЬ
Ну это еще не весь список там после этого еще можно было бы улучшить
N
Ну это еще не весь список там после этого еще можно было бы улучшить
Что ищё?
AЬ
Null
Что ищё?
Деструктуризация вызывает вопросы, вангую баги будущие
N
Ну просто сейчас удобно вроде
AЬ
Тапл можно в тайп отдельный локальный приватный вынести
AЬ
Мутация данных тоже возможно лишняя
AЬ
И сама идея того что мэп юзается для сайд эффектов
AЬ
У вас выглядит так что сам мэп никак данные напрямую не возвращает хотя именно для этого он и нужен
AЬ
кстати, а если, допустим есть тостер и там для показа тоста надо подписаться, а в сабскрайбе делать как бы нечего, как в таком случае поступить?
Код бы желательно увидеть
N
Внутри combineLatest изменил на tap
N
Мутация данных тоже возможно лишняя
Про эту мутацию?
this.accountData[0].personal_data = recipientId;AЬ
this.accountData непонятно почему как массив, возможно его лучше вообще как $
AЬ
Null
Про эту мутацию?
this.accountData[0].personal_data = recipientId;Da
AЬ
И сам this.accountData судя по манипуляциям с ним возможно не совсем массивом должен быть
AЬ
Ну не то чтобы должен но возможно массив там не совсем подходит, но из контекста не на 100% понятно
2020 October 01
N
Ну не то чтобы должен но возможно массив там не совсем подходит, но из контекста не на 100% понятно
Да спасибо, я сделал объект и потом в шаблоне object.values