msenyk / sf-practicum-2022

Виробнича практика на платформі Salesforce
0 stars 0 forks source link

Розробити компонент для генерації турів #6

Open msenyk opened 2 years ago

msenyk commented 2 years ago
20sergejka02 commented 2 years ago

Компоненты на странице(форме)

Пример:

1
msenyk commented 2 years ago

@20sergejka02 Давайте домовимось називати поля англійською, щоб все було в одному стилі. Ще на формі можна додати поле (тільки для читання) яке б показувало скільки таких турів буде сгенеровано за період.

msenyk commented 2 years ago

@20sergejka02 Як назвивається компонент?

foxxua commented 2 years ago

@20sergejka02 Є декілька зауважень

  1. назву компонента testLWC потрібно змінити на щось інше, більш предметне
  2. Я раджу почитати стандарти SLDS для того щоб привести зовнішній вигляд компоненту (UI) до кондиції Переношу таску в In Progress
20sergejka02 commented 2 years ago

Ок, исправлю

20sergejka02 commented 2 years ago

Проблему решил, спасибо

msenyk commented 2 years ago

@20sergejka02 Щось не так із валідацією. Кроки для відтворення:

  1. Ввести дату початку 20 січня
  2. Ввести дату кінця 17 січня
  3. Натиснути Generate
  4. Бачу повідомлення про помилку, що вірно
  5. Міняю дату кінця на 21 січня
  6. Натискаю Generate
  7. Знову бачу повідомлення валідації, а це не вірно
20sergejka02 commented 2 years ago

Нужно ещё раз нажать чтобы программа обновилась или код -- это тоже не очень понимаю.

msenyk commented 2 years ago

@20sergejka02 Там не правильно реалізовано логіку валідації. Помилка в методі showNotification() тут:

    else if(this.result < 0)
    {
        const evt = new ShowToastEvent({title: "Enter a valid value", message: "Enter a start date no earlier than today, and an end date no later than today.", variant: "Error"});
        this.dispatchEvent(evt);
    }

    else
    {
        this.result = this.date2 - this.date1;

Як видно розрахунок значення result відбувається у блоці else, а то вже запізно. Бо виконання перехватує else if.

20sergejka02 commented 2 years ago

Есть

20sergejka02 commented 2 years ago

Сделал поля обязательными, и ограничил ввод дат с 01-01-2020 до 01-01-2025