ну вот если бы хендлеры .then() возвращали промисы/значения — можно было бы всё в 1 цепочку объединить без вложенности, и в конце поставить один на всех .catch()

хмм, ну один catch у меня возвращает уже 401, так что уже -1

всем привет, может не по теме, но кто нибудь может посоветовать хост с впс, где есть возможность оплаты битками?

всем привет, может не по теме, но кто нибудь может посоветовать хост с впс, где есть возможность оплаты битками?

https://poiskvps.ru/index.php?search_payment_method%5B%5D=9

https://poiskvps.ru/index.php?search_payment_method%5B%5D=9

не правильно вопрос задал наверное, посоветовать\порекомендовать проверенный надежный хост

Привет, кто то юзал nest.js на практике? Как впечатления?

Привет, кто то юзал nest.js на практике? Как впечатления?

я не юзал, но читал доки, и только с них впечатлился

Привет, кто то юзал nest.js на практике? Как впечатления?

Сначала примеры кода чекни

Если ЭТО тебя устраивает, то норм

Примеры волшебные, ищу подвох)

это нормально или я упоролся?

навреное, вот так наиболее правильно будет reminders - lazy load - performace

Как правильнее назвать роут: /sign-up Или /signup ??

это нормально или я упоролся?

так тоже норм, только без отрицания в условии тернарного оператора

а uuid - везде одна переменная?

тяжко без async/await

что-то пошло не так

+++

навреное, вот так наиболее правильно будет reminders - lazy load - performace

а зачем «let reminders»? значение же только в одном месте используется? откуда тут lazy load?

ну типа, вроде так

объясни что за lazy load и как он тут по-твоему работает?

это нормально или я упоролся?

вот этот вариант мне нравится больше всего

объясни что за lazy load и как он тут по-твоему работает?

я помоему, не правильно понял https://firebase.google.com/docs/functions/tips?authuser=0#do_lazy_initialization_of_global_variables

ладно, хватит мусолить эту телегу)

я помоему, не правильно понял https://firebase.google.com/docs/functions/tips?authuser=0#do_lazy_initialization_of_global_variables

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

никак не могу найти один веб-инструмент, который видел где-то полгода назад. он на английском языке, выглядит как дерево, построенное по API стандартной библиотеки JS (или может по документации языка, не помню). в этом дереве можно отметить какие разделы ты знаешь, узлы дерева можно сворачивать/разворачивать. больше ничего не помню. никто не встречал такое? не выгуглить никак :(

тяжко без async/await

Вложенные then это антипаттерн. Возвращай промис из then, а не вызывай новый. Или это типа шутка?)

Вложенные then это антипаттерн. Возвращай промис из then, а не вызывай новый. Или это типа шутка?)

не шутка

Тогда все плохо

не шутка

У тебя какой-то callback hell, только из промисов)) Нужно возвращать из каждого then следующий промис, как сказал Danilian

У тебя какой-то callback hell, только из промисов)) Нужно возвращать из каждого then следующий промис, как сказал Danilian

точно, только сейчас осознал, всё время юзаю await, что аж забыл

Перепиши и покажи нам новый вариант)

В идеале у тебя должен быть красивый chain из этих промисов и всего 1 catch

CodeReview от анонов в чатике

"Жизнь без тимлида и коллег, а ревью нужно"

CodeReview от анонов в чатике

ахаах

ну вот вроде так

только вот catch будет вызываться не на последнем лишь промисе?

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

оу май гад, вот это круто

Не уверен что в последнем then тебе надо делать return

там это не нужно имхо

в firebase нужно что бы firebase function возвращала промис или просто ретурнилась

так что вроде норм

Ты уверен кстати что auth().getUser(uid) вернет тебе в итоге объект пользователя с полем uid?

Который ты потом в database().ref() используешь

getUser(uid) returns Promise containing non-null admin.auth.UserRecord - я так понял, либо возвращает UserRecord, либо reject-ится

в UserRecord поле uid есть?

+

Ну тогда окей

Кажись красивее стало

ну всё)

спасибо за ревью :)

Только ты проебал ;

Где ; ????!!!!

а зачеееем?

Ну это тебе не питон, надо ; =(

и тут наши пути разошлись :D

Так стой, я знаю как еще -1 строчку сделать

Ну это тебе не питон, надо ; =(

Не надо

res.json(userSnap)

Или userSnap это моделька на самом деле а не объект?

+

и в нем toString говно

А ну тогда увы =/

(((

toJson вернет строку?

или объект

объект

тогда лучше res.json, он ведь заголовки выставляет

Вру

С объектами пофиг

вроде да

express умный и сам знает какой заголовок для объектов делать

вот так выглядит вот это вот без .toJSON()

express умный и сам знает какой заголовок для объектов делать

Коа еще умнее

Коа еще умнее

Уходи

эти express и koa дрочеры

Уходи

А еще есть fastify

а еще есть polka

ахаха