Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix-festival-event-card #569

Merged
merged 3 commits into from
Sep 3, 2024
Merged

Conversation

MADeit0
Copy link
Collaborator

@MADeit0 MADeit0 commented Sep 1, 2024

Описание

Этот Pull Request исправляет ошибку при которой в афише у спектакля отсутствовала ссылка для перехода на страницу спектакля.
При наведении на заголовок спектакля добавлена визуальная подсказка, что элемент интерактивный.

Снимок экрана 2024-09-01 в 09 44 44

Ссылка на задачу

Проблема с отображением ссылок на спектакли

@MADeit0
Copy link
Collaborator Author

MADeit0 commented Sep 1, 2024

Единственно не совсем понял как по структуре проекта заимствовать css свойства

@MADeit0
Copy link
Collaborator Author

MADeit0 commented Sep 1, 2024

И может у меня плохо установились зависимости, но я не смог в дэв режиме открыть ссылку по пути http://localhost:3000/performances/10
Снимок экрана 2024-09-01 в 09 58 58

@FinemechanicPub
Copy link
Collaborator

И может у меня плохо установились зависимости, но я не смог в дэв режиме открыть ссылку по пути http://localhost:3000/performances/10 Снимок экрана 2024-09-01 в 09 58 58

Если я правильно помню, локально развернутый frontend обращается к демоверсии API stage.dev.lubimovka.ru. Там 10-й спектакль был не опубликован. Теперь я его опубликовал для удобства.

@@ -73,6 +73,7 @@ function mapDTOToFestivalEvents(dto: EventsDTO): PaginatedFestivalEvents {
date: event.date_time,
registrationOpeningDate: event.opening_date_time,
registrationUrl: event.action_url,
aboutURL: event.performance_id
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Выглядит нехорошо. Это не URL, а число. Получается, в разных местах aboutURL обозначает разные вещи.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

performance_id

Поле aboutUrl в компоненте FestivalEventCard предназначено для отображения ссылки на страницу выступления
@AntonZelinsky AntonZelinsky merged commit 10c6137 into develop Sep 3, 2024
1 check 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.

Проблема с отображением ссылок на спектакли
3 participants