fix: add allowAvatar prop to task-header component for conditional av… - #1610
Conversation
|
|
||
| const withAvatar = computed( | ||
| () => props.size === ComponentSize.SM && !!props.username, | ||
| () => props.size === ComponentSize.SM && props.allowAvatar, |
There was a problem hiding this comment.
не розумію, чого воно не робе, враховуючи що ти в пропс новий дописав буквально те ж саме, що є в пропсі username
а шо там не так?
There was a problem hiding this comment.
воно то робе, але якщо передавати юзернейм як Unknown Contact, то аватарка відображається як UC, а треба N/A, посилання на комент - https://webitel.atlassian.net/browse/WTEL-10421?focusedCommentId=785599
There was a problem hiding this comment.
всеодно недоганяю.... мені вже тупо цікаво розібратися)))
| @@ -1,5 +1,5 @@ | |||
| <template> | |||
| <task-header :size="props.size" :username="call?.contact ? title : undefined"> | |||
| <task-header :size="props.size" :allow-avatar="!!call?.contact" :username="call?.contact ? displayName : undefined"> | |||
There was a problem hiding this comment.
отут, (якщо без нового пропса) якщо немає контакту, то вертається андерфайнд, а значіть аватар не покажеться
| @@ -1,5 +1,5 @@ | |||
| <template> | |||
| <task-header :size="props.size" :username="chatInfo.username"> | |||
| <task-header :size="props.size" :allow-avatar="!!chatInfo.username" :username="chatInfo.username"> | |||
There was a problem hiding this comment.
тут (якщо без нового пропса), по-суті !!chatInfo.username перевірка йде в компоненті самому отут https://github.com/webitel/cc-workspaces/pull/1610/changes#diff-f0c32be5b0b386dbffe3b60e00b309ebf8149d136ffe37d4079f3644bfe37bb4L34
якщо ж З новим пропсом, то якщо в chatInfo.username значення якесь тіпа "Unknown Contact", то allow-avatar буде тру
| @@ -1,5 +1,5 @@ | |||
| <template> | |||
| <task-header :size="props.size" :username="task.displayName"> | |||
| <task-header :size="props.size" :allow-avatar="!!task.displayName" :username="task.displayName"> | |||
🤖 Webitel Code ReviewЦей комміт додає нову властивість 📋 Walkthrough (2 файл(и/ів))
Знахідки
Index-grounded review across the Webitel codebase. Знахідки можуть бути неточними — перевіряйте перед застосуванням. |
![webitel-review[bot]](https://avatars.githubusercontent.com/in/4658759?s=60&v=4){kind=small}
| const withAvatar = computed( | ||
| () => props.size === ComponentSize.SM && !!props.username, | ||
| () => | ||
| props.size === ComponentSize.SM && (props.allowAvatar || !!props.username), |
There was a problem hiding this comment.
Використання оператора ?? дозволить враховувати явне передане значення false для allowAvatar, а за його відсутності (undefined) повертатися до перевірки наявності username.
| props.size === ComponentSize.SM && (props.allowAvatar || !!props.username), | |
| props.size === ComponentSize.SM && (props.allowAvatar ?? !!props.username), |
2 participants
…atar display WTEL-10421