Skip to content

4.21. Обратная сторона реальности#4

Merged
AlSudar merged 12 commits intohtmlacademy-react:masterfrom
alisameta09:module4-task1
Jun 5, 2025
Merged

4.21. Обратная сторона реальности#4
AlSudar merged 12 commits intohtmlacademy-react:masterfrom
alisameta09:module4-task1

Conversation

@alisameta09
Copy link
Copy Markdown
Contributor

4.21. Обратная сторона реальности

@alisameta09
Copy link
Copy Markdown
Contributor Author

2025-06-02_10-44-26
2025-06-02_10-44-49
2025-06-02_11-53-11
2025-06-02_11-53-28

Добрый день, у меня как всегда куча ошибок и я без понятия, что с ними делать. С дз и так оч долго разбиралась, надеюсь вы сможете мне помочь все это пофиксить и заставить работать. И хорошо бы еще я поняла причину и как это исправлять)

вопросы:

  1. Почему у меня куча замечаний от линтера ( на первом скрине видно и как это исправить)?
  2. Почему у меня не сработало состояние disabled на стр review-form по настройкам в презентации, а не ломает страницу только с вопросительными знаками и в таком виде, но все равно работает некорректно?
  3. Некорректно работает проверка isAuth в offer-page, тоже не понимаю почему и вообще правильно ли я все настроила, не понимаю какое состояние передается в эту проверку и откуда оно берется, если честно.
  4. Попыталась сделать один header по вашему совету, но немного на свое усмотрение, используя то же состояние isAuth и частично получилось, но не отображается кусочек, кот я выделила с данными пользователя, когда он залогинен. Подскажите, как сделать, чтобы header заработал корректно и как проверить работает ли он корректно при незарегистрированном статусе пользователя.
  5. Также не знаю как правильно сделать переключение активного и неактивного оффера в place-card, пока сделала как у всех: <button className={place-card__bookmark-button button ${offer.isFavorite && 'place-card__bookmark-button--active'}} type="button">, но у меня есть отдельный компонент под это: place-card-bookmark и хотелось бы задействовать его, но что бы я не пробовала - он у меня не заработал корректно. Надеюсь на вашу помощь. Спасибо.

@AlSudar
Copy link
Copy Markdown
Contributor

AlSudar commented Jun 2, 2025

  1. Почти везде он ругается на то, что тип данных может быть любым. Нужно указать конкретный тип данных
  2. Потом что reviews может быть undefined
  3. В offerPage сейчас путаница. В качестве пропсов туда нужно передавать authorizationStatus (по крайней мере у тебя так типизировано), но почему то передаем reviews. AuthorizationStatus у тебя на эту страницу просто импортируется снаружи (7 строка)
  4. Нужно передавать в isAuth значение при вызове Header, в зависимости от этого значения условия в коде будут по разному себя вести.
  5. Нужно добавить в button который находится в PlaceCardBookmark условие во внутрь класса. Примерно так:
return (
    <button className=`place-card__bookmark-button button ${offer.isFavorite && 'place-card__bookmark-button--active` type="button">
      <svg className="place-card__bookmark-icon" width="18" height="19">
        <use xlinkHref="#icon-bookmark"></use>
      </svg>
      <span className="visually-hidden">To bookmarks</span>
    </button>
  );

@alisameta09
Copy link
Copy Markdown
Contributor Author

  1. Почти везде он ругается на то, что тип данных может быть любым. Нужно указать конкретный тип данных - указала через as string, можно так? Лучше способа я не вспомнила, а времени пересматривать блок про типы у меня совсем нет, иначе отстану еще сильнее. Можете показать пример, пожалуйста.
  2. Потом что reviews может быть undefined - можете показать как, не выходит. Целый день искала сегодня решение, не понимаю как прописать тип в данном случае, все время ломается, что-то другое.
  3. Нужно добавить в button который находится в PlaceCardBookmark условие во внутрь класса. Примерно так: ... - пробовала разными способами и так в том числе - все равно не работает корректно. Можете проверить и написать мне поточнее, к сожалению, не понимаю в чем дело.

С остальным удалось разобраться, спасибо за подсказки.

Не знаю, что делать с build, можете показать как решить текущие ошибки? Остальные я решила полотнище было целое, а эти не оч понимаю как. Спасибо большое, что так помогаете.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 4, 2025

Ваш пулреквест опубликован. Посмотреть можно здесь

github-actions bot pushed a commit that referenced this pull request Jun 4, 2025
@AlSudar AlSudar merged commit 5d2be59 into htmlacademy-react:master Jun 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants