search

msenyk / sf-practicum-2022

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

Доопрацювати компонент генерації турів для відображення кількості турів, яку ми плануємо сгенерувати #17

Open msenyk opened 2 years ago

msenyk commented 2 years ago

Під час зміни діапазону дат треба показувати на формі кількість турів що має бути сгенерована. Наприклад, якщо в нас тур на 2-чи на тиждень, а ми обрали діапазон 3 тижні, то у формі маємо відобразити число 6.

20sergejka02 commented 2 years ago

Не получается достать recordId тура.

Пытался вот так: this.recordId = document.URL.split('/')[6] , но Вы забраковали)

Гуглил -- не нашёл. Помогите пожалуйста.

msenyk commented 2 years ago

@20sergejka02 Ти маєш дістати Id не тура, а типу екскурсії. Для цього в СФ є стандартний спосіб - https://developer.salesforce.com/docs/component-library/documentation/en/lwc/use_record_context

msenyk commented 2 years ago

@20sergejka02

  1. Чомусь перевірка діапазону дат почала працювати не вірно. Наприклад, якщо обрати початок 24-го, а кінець 29-го, але потім змінити кінець на 28-є, то пише Difference of days: -1.
  2. Трохи не так реалізовано дату початку та закінчення. Наприклад, якщо у екскурсії 2 дні - четвер та п'ятниця, і я вів діапазон з вівторка по четвер - то це 1 день, бо четвер потрапляє в діапазон, а п'ятниця - ні.
  3. Вміст методів stDate(event) та fnDate(event) майже ідентичний. Краще винести спільну логіку в окремий метод і викликати його з цих методів.
  4. В циклі використовується властивість компоненту this.emptyDate. Не те що так не можна робити, але який в цьому смисл? То ж має бути локальна тимчасова змінна.
  5. JavaScript на жаль не дуже вправно працює з датами. Ось посилання, що може трохи спростити операцію додавання одного дня до поточної дати: https://attacomsian.com/blog/javascript-date-add-days
msenyk commented 2 years ago

@20sergejka02 Визначення кількості турів працює вірно, молодець. Але чомусь перестала працювати генерація їх на сервері. Можеш перевірити?

20sergejka02 commented 2 years ago

Хм, странно, всё работало. Я не знаю в чём может быть причина, всё проверил, по идее должно всё работать...

msenyk commented 2 years ago

@20sergejka02 добре, тоді можна доопрацювати зв'язок із серверною логікою у розрізі #16