fix: limited audit form score inputs [WTEL-4505] #231
Conversation
…dded limitation for score inputs, fixed tests [WTEL-4505]
Qodana for JSIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
| @@ -16,7 +16,7 @@ describe('AuditFormQuestionOptionsWriteRow', () => { | |||
| option: {}, | |||
| }, | |||
| }); | |||
| const deleteBtn = wrapper.findComponent({ name: 'wt-icon-btn' }); | |||
| const deleteBtn = wrapper.findComponent('.audit-form-question-options-write-row__tooltip').findComponent({ name: 'wt-icon-btn' }); | |||
There was a problem hiding this comment.
Мені ось тут не дуже подобається findComponent 2 рази(
There was a problem hiding this comment.
Згодна(( вчора шукала інші способи, вивчала доку, пробувала.
Питання в тому, що там треба знайти іконку в конкретному блоці. І я навть думала потрібній іконці псотавити просто клас, шоб шукати по ньому. Але, блін, цей клас не потрібен для стилів. Додавати лише для тестів звучить як фіговий варіант...
Спробую знайти, як це можна скоротити
There was a problem hiding this comment.
Оце я гоню, звісно...там же просто роботою з селекторами можна порєшать..
| @@ -72,6 +75,11 @@ const v$ = useVuelidate( | |||
| { $autoDirty: true }, | |||
| ); | |||
|
|
|||
| function changeScore (value) { | |||
| const score = value > 10 ? 10 : Number(Math.abs(value)); // to prevent -1, 000 or string value | |||
There was a problem hiding this comment.
Я би додала посилання на таску, бо невідомо звідки саме 10 взялось
- 10 - винести у веріаблу
Але може це не обовязково, на твій розсуд
There was a problem hiding this comment.
слушно, дякую!
src/modules/AuditForm/components/questions/options/audit-form-question-options-write-row.vue
Show resolved
Hide resolved
| @@ -111,7 +111,7 @@ const scoreRange = computed(() => { | |||
| const isResult = computed(() => !isEmpty(props.result)); | |||
|
|
|||
| function updateQuestion({ path, value }) { | |||
| const number = value > 10 ? 10 : Number(Math.abs(value)); // to prevent -1, 000 or string value | |||
There was a problem hiding this comment.
тут все складніше, бо тут має буть по 2 змінін на кожен інпут.
власне, тому я і не стала тут робити.
маю на увазі, що там для одного інпута значення min - 0, а для іншого - 1
так само і з max. для одного - 9, а для іншого - 10
мені здається, що з цим вже варто морочитись під час виправлення валідації саме для цих інпутів.
Маю на увазі, що там немає валідації, якщо інпут from більше ніж to.
src/modules/AuditForm/components/questions/options/audit-form-question-options-write-row.vue
Show resolved
Hide resolved
| @@ -109,7 +111,8 @@ const scoreRange = computed(() => { | |||
| const isResult = computed(() => !isEmpty(props.result)); | |||
|
|
|||
| function updateQuestion({ path, value }) { | |||
| emit('change:question', updateObject({ obj: props.question, path, value })); | |||
| const number = value > 10 ? 10 : Number(Math.abs(value)); // to prevent -1, 000 or string value because of this task https://webitel.atlassian.net/browse/WTEL-4505 | |||
There was a problem hiding this comment.
а чому так складно?
чому не перевірити чи намбер, і в межах 0-10?
There was a problem hiding this comment.
Бо я роблю трошечки фінт ушамі і підставляю 10, якщо людина вводить цифру більше 10.
мені це дуже подобається) як в монобанку 😁😁😁
No description provided.