search

msenyk / sf-practicum-2022

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

Дозволити адміністратору конфігурувати кількість турів в компоненті Up Next #25

Open msenyk opened 2 years ago

msenyk commented 2 years ago

Зараз кількість турів, що відображаються в компоненті жорстко обмежена кількістю 5. Треба передбачити можливість змінювати цю кількість. Бажано, щоб це не призводило до повторної публікації сайту.

msenyk commented 2 years ago

@thxnk Custom Metadata Type обрано вірно для зберігання кофігурації. Але є деякі недоліки, що варто виправити:

  1. Назва типу (Display UpNext Tours) обмежує застосування його тільки компонентом Up Next. Можливо треба використати це налаштування ширше. Наприклад, Site Settings назва дозволить тримати всі налаштування щодо сайту в одному місці.
  2. Що буде якщо адміністратор не створить жодного запису або створить 2?
  3. Варто передбачити значення за умовчанням, якщо налаштування відсутні або введене некоректне значення
  4. Методи класу TourController getTours() та getLimitTours() мають майже однакові SOQL запити. Варто подумати як винести той виклик у окремий приватний метод.