Skip to content
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.

feat/landing-4 #18

Merged
merged 9 commits into from
May 24, 2023
Merged

feat/landing-4 #18

merged 9 commits into from
May 24, 2023

Conversation

oxiqod
Copy link
Contributor

@oxiqod oxiqod commented May 19, 2023

No description provided.

@oxiqod oxiqod self-assigned this May 19, 2023
@oxiqod oxiqod linked an issue May 19, 2023 that may be closed by this pull request
@oxiqod
Copy link
Contributor Author

oxiqod commented May 23, 2023

@TorinAsakura

Скоуп
добавление слайдера

Описание
нужно добавить слайдер в фрагмент courses

Воспроизведение

  1. смотрю примеры и по аналогии добавляю в ui новый компонент
  2. создаю пакедж @ui/slider и ставлю зависимости
  3. запускаю yarn, получаю ответ в поле Link step - swiper@npm:8.3.1 must be built because it never has been before or the last one failed
  4. запускаю yarn check - получаю ошибку Typecheck -
    .yarn/unplugged/swiper-npm-8.3.1-7575c41fae/node_modules/swiper/react/swiper-react.d.ts
    Error: Cannot find module 'react' or its corresponding type declarations.

Вопрос
Из-за чего эта ошибка?

@TorinAsakura
Copy link
Member

@oksssvv 3. Это нормальный процесс линковки, почитай уже про работу зависимостей во втором ярне
4. Ошибка тебе явно говорит откуда она растёт, у тебя в корне свайпер анплагнут, в котором, не может найтись реакт. Не?

@oxiqod
Copy link
Contributor Author

oxiqod commented May 24, 2023

@TorinAsakura

Скоуп
Создание слайдера в мобильной версии

Описание
Был создан слайдер, рендер отрабатывает как надо и слайдер работает, но при проверке type check - выдает ошибку

Воспроизведение

  1. создан ui компонент Slider (проверка молчит)
  2. когда в фрагменте создаю импорт и расставляю компоненты Swiper и SwiperSlide - проверка проваливается

Логи
.yarn/unplugged/swiper-virtual-7cbc08c7b0/node_modules/swiper/react/swiper-react.d.ts Erro: Interface 'SwiperProps' incorrectly extends interface 'Omit<HTMLAttributes<HTMLElement>, "onKeyPress" | "onProgress" | "onClick" | "onDoubleClick" | "onTouchEnd" | "onTouchMove" | "onTouchStart" | "onScroll" | "onTransitionEnd">'. Types of property 'onResize' are incompatible. Type '((swiper: Swiper) => void) | undefined' is not assignable to type 'ReactEventHandler<HTMLElement> | undefined'. Type '(swiper: Swiper) => void' is not assignable to type 'ReactEventHandler<HTMLElement>'. Types of parameters 'swiper' and 'event' are incompatible. Type 'SyntheticEvent<HTMLElement, Event>' is missing the following properties from type 'Swiper': params, originalParams, $el, el, and 90 more.

.yarn/unplugged/swiper-virtual-7cbc08c7b0/node_modules/swiper/react/swiper-react.d.ts Error: Interface 'SwiperSlideProps' incorrectly extends interface 'HTMLAttributes<HTMLElement>'. Types of property 'children' are incompatible. Type 'ReactNode | ((slideData: SlideData) => ReactNode)' is not assignable to type 'ReactNode'. Type '(slideData: SlideData) => ReactNode' is not assignable to type 'ReactNode'.

Действия

  1. гуглила
  2. пробовала менять различные завимости и версии в пакете ui и в компоненте который впоследствии рендерится
  3. пробовала добавить некоторые зависимости в .yarnrc.yml

Вопрос
Почему может выбрасывать эту ошибку?

@TorinAsakura
Copy link
Member

@oksssvv Он написал почему: у тебя представленный тип не соответствует содержанию.

@TorinAsakura
Copy link
Member

@oksssvv nolimits4web/swiper#5799

@oxiqod oxiqod requested a review from TorinAsakura May 24, 2023 10:55
@TorinAsakura TorinAsakura merged commit 9cadd9b into master May 24, 2023
@TorinAsakura TorinAsakura deleted the feat/landing-4 branch May 24, 2023 18:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Academy landing, step 4
2 participants