Issue #130: Грейд версий @atls/react-kratos-browser-flows #132
Conversation
| "@types/react": "18.2.20", | ||
| "@types/react-dom": "18.2.7", | ||
| "@types/testing-library__jest-dom": "^5.14.1", | ||
| "@testing-library/jest-dom": "^5.17.0", |
There was a problem hiding this comment.
^ в версиях не используем, только жестко фиксированные как ты сделал с @types/react
| "@types/react": "18.2.20", | ||
| "@types/react-dom": "18.2.7", | ||
| "@types/testing-library__jest-dom": "^5.14.1", | ||
| "@testing-library/jest-dom": "^5.17.0", |
There was a problem hiding this comment.
Выше версии не обновлены, особенно @ory/kratos-client
There was a problem hiding this comment.
если обновить @ory/kratos-client до последней версии, то возникают ошибки. Пример на скрине:
Details
Наиболее последняя версия, с которой ошибок нет - 0.9.0-alpha.3. Оставлять ее?
There was a problem hiding this comment.
Попробуй добавь в зависимости пакета:
"@ory/client": "1.2.11",
"@ory/integrations": "1.1.5",
"@ory/kratos-client": "1.0.0",There was a problem hiding this comment.
Добавил. Ошибки есть. Пример на скрине.
Details
There was a problem hiding this comment.
Замени в проекте:
SubmitSelfServiceVerificationFlowBodyнаVerificationFlowBody
По остальному посмотри откуда идут экспорты тут https://github.com/atls/nextjs/tree/master/packages/identity-integration
Это аналогичный пакет, но под nextjs
There was a problem hiding this comment.
Замени в проекте:
SubmitSelfServiceVerificationFlowBody на VerificationFlowBody
Ошибка:
packages/kratos-browser-flows/src/flows/verification.flow.ts
➤ YN0000: │
➤ YN0000: │ Error: Module '"@ory/kratos-client"' has no exported member 'VerificationFlowBody'.
There was a problem hiding this comment.
Проверься с тем пакетом что выслал - там тот же функционал и экспорты. Из-за старости версии тут могут различаться по названию интерфейсов.
| "@types/events": "^3.0.2", | ||
| "@types/react": "18.2.31", | ||
| "@types/react-dom": "18.2.14", | ||
| "@types/testing-library__jest-dom": "^5.14.9", |
There was a problem hiding this comment.
peerDependencies так же не обновлены
| "@emotion/react": "11.9.3", | ||
| "eslint": "8.18.0", | ||
| "typescript": "4.6.2" | ||
| "@emotion/react": ">=11.9.0 <11.10.0", |
There was a problem hiding this comment.
Тут я тебя ввел в заблуждение - обычно когда говорим "не выше" имеется в виду что можно использовать и последнюю. Т.е. @types/react я бы зафиксировал "17.Х.Х", где Х - последняя версия на npmjs.com
@emotion/react, eslint и typescript здесь лишние - их мы можем фиксировать вручную в проектах.
Вообще resolutions используется тогда, когда наши внешние зависимости тянут с собой что то, что нас не устраивает по версии. Вот твой случай - у тебя из-за внешних зависимостей @types/react "задваивался", из-за чего были ошибки. Поэтому я предложил тебе воспользоваться resolutions
There was a problem hiding this comment.
@emotion/react, eslint и typescript здесь лишние - их мы можем фиксировать вручную в проектах.
Имеешь в виду, что @emotion/react, eslint и typescript можно убрать из resolutions? Они были там до меня.
There was a problem hiding this comment.
Можешь оставить, главное версии держи четко зафиксированными. Ну и посмотри чтобы не было диссонанса - например, в resolutions тайпскрипт - 4.6.4, а ниже - 4.6.2
No description provided.