Skip to content

Добавить рантайм проверку src в useImageLoadingState #83

Open
koretskyhub opened this issue Jan 21, 2022 · 0 comments

Comments

@koretskyhub
Copy link
Contributor

koretskyhub commented Jan 21, 2022

Предлагаю подумать над тем чтобы добавить рантайм проверку src в useImageLoadingState, проверка типами не всегда может помочь увидеть проблему

Например, тк у нас в половине банка тип null в принципе на деле не используется, возможно что ts пропустит src === null как аргумент.
Недавно в новом клике был прецедент, в проде получили кучу лишних запросов за картинкой.
Кажется, что лучше добавить рантайм проверку в хук, чтобы наверняка избежать лишних запросов когда по какой-то причине в хук уходит неадекватная строка (пустая или null)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants