-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat(plasma-new-hope): Modal added #828
Conversation
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
84a0bd9
to
5386078
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
5386078
to
3d8af2e
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
3d8af2e
to
9930d7b
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
if (!modals.length) { | ||
return; | ||
} | ||
return modals[modals.length - 1].id; |
Check warning
Code scanning / Semgrep
Semgrep Finding: gitlab.eslint.detect-object-injection Warning
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
packages/plasma-new-hope/src/examples/plasma_b2c/components/Modal/Modal.config.ts
Outdated
Show resolved
Hide resolved
packages/plasma-b2c/src/components/ModalBase/ModalBase.stories.tsx
Outdated
Show resolved
Hide resolved
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
b5de9ce
to
5900137
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
5900137
to
f247d98
Compare
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
⚡ Component performance testingResult: 🟢 OK |
6f868e8
to
e8e0e6e
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
⚡ Component performance testingResult: 🟢 OK |
e8e0e6e
to
7652350
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
7652350
to
2a61695
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-828/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-828/ |
⚡ Component performance testingResult: 🟢 OK |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
4 similar comments
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
Добавлен компонент
Modal
вplasma-new-hope
.Переведены компоненты
ModalBase
,PopupBase
вweb, b2c
наnew-hope
(также они удалены изplasma-hope
).⚡ Component performance testing
Result: 🟢 OK
⚡ Component performance testing
Result: 🟢 OK
🐤 Download canary assets:
📦 Published PR as canary version:
Canary Versions
✨ Test out this PR locally via:
Version
Published prerelease version:
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
@salutejs/[email protected]
Changelog
Release Notes
plasma-infra: Migrate to storybook 7 [Web] (#850)
Переход на
Storybook@latest(7.5.3)
:What/why Changed
From version 6.5.x to 7.0.0
.stories
to CSF 3main.ts
preview.ts
plasma-infra(audit): Fix critical several vulnerability [plasma-hope] (#846)
Исправлены critical several vulnerability.
What/why Changed
Before::
npm audit
After::
npm audit
Before:: Dependabot alerts
plasma-infra: Migrate to Storybook 7 [B2C] (#847)
Переход на
Storybook@latest(7.5.3)
:What/why Changed
From version 6.5.x to 7.0.0
.stories
to CSF 3main.ts
preview.ts
plasma-infra(new-hope): Migrate to Storybook 7 (#845)
Переход на
Storybook@latest(7.5.3)
:What/why Changed
From version 6.5.x to 7.0.0
.stories
to CSF 3 (Более подробное описание - plasma-infra(b2c): Migrate to CSF 3 [Migrate SB 7 Part 1] #833)main.ts
preview.ts
plasma-infra(b2c): Migrate to CSF 3 [Migrate SB 7 Part 1] (#833)
Файлы
*.stories
переведены на новый формат, необходимый для перехода на Storybook 7What/why Changed
Для того что бы перейти на Storybook 7 нужно переписать на новый формат все истории.
Основные изменения в том что ориентируются на ESM и теперь
storie
- это объект, описывающий состояние компонента. В целом такой формат позволяет гибче выстраивать композицию и share code.Before (CSF 2)
After (CSF 3)
Обо всех изменениях можно прочитать Component Story Format 3 is here
⚡ Component performance testing
Result: 🟢 OK
⚡ Component performance testing
Result: 🟢 OK
plasma-infra: Bump [email protected] (#843)
Переход на версию [email protected]
What/why Changed
Чтобы безболезненно перейти на Storybook 7 нужно обновить typescript.
Иначе придется ставить ряд пакетов подтипу
@types/package@
и версия под[email protected]
Сборка и использование не выявили регресса.
https://github.com/microsoft/TypeScript/wiki/Breaking-Changes#typescript-42
ci: write report comment instead of writing to PR description (#838)
Отчет перфтестов писать в комментах, а не в шапке PR, как было ранее.
Тест в #839
What/why Changed
Писать отчет в шапку PR оказалось плохой идеей, поскольку технически сложно гарантировать отсутствие коллизий с release джобой. Синхронизация затруднительна из-за того, что мы поддерживаем не весь код, который пишет в шапку.
plasma-ASDK: Add new package "ASDK" (#835)
Plasma-asdk:
Plasma-new-hope:
Добавлена типографика в виде компонент как старой, так и новой типографики, а так же в виде mixins.
plasma-infra: Add
.editorconfig
(#837)Добавлен:
.editorconfig
package.json
в.prettierignore
What/why Changed
Что бы локальные настройки IDE форматирования не применялись для файлов проекта.
Для консистентного форматирования добавлен
.editorconfig
и для всехpackage.json
сделано форматирование с учетом отступа в 4 пробела.Так же добавлено правило
.prettierignore
дляpackage.json
, что быlintstaged
не применял для него форматирование.⚡ Component performance testing
Result: 🟢 OK
plasma-infra(audit): Resolve root dependencies(dev) vulnerabilities (#824)
Почищен корневой package.json:
What/why Changed
Storybook удален так как является рудиментом от установки/использования библиотеки chromatic.
Resolve transitive dev dependencies
Для корректного запуска perf test:
feat(plasma-new-hope): Add calendar with new architecture (#821)
Добавлена реализация компонента Calendar в новой архитектуре.
Добавлены сборки компонента для новых тем plasma_web, plasma_b2c, sds_engineer.
Добавлены сборки компонента для старых библиотек @salutejs/plasma-web, @salutejs/plasma-b2c.
Удалён компонент Calendar из @salutejs/plasma-hope.
Поправил небольшой баг (issue) с типизацией для mergeConfig: если у компонента были обязательные пропсы ts ругался, что их не хватает в самом конфиге.
fixes: #597
⚡ Component performance testing
Result: 🟢 OK
⚡ Component performance testing
Result: 🟢 OK
⚡ Component performance testing
Result: 🟢 OK
⚡ Component performance testing
Result: 🟢 OK
⚡ Component performance testing
Result: 🟢 OK
🚀 Enhancement
@salutejs/plasma-b2c
,@salutejs/plasma-new-hope
,@salutejs/plasma-web
@salutejs/plasma-b2c
,@salutejs/plasma-hope
,@salutejs/plasma-new-hope
,@salutejs/plasma-web
@salutejs/plasma-new-hope
@salutejs/plasma-asdk
,@salutejs/plasma-b2c
,@salutejs/plasma-core
,@salutejs/plasma-hope
,@salutejs/plasma-web
@salutejs/plasma-asdk
,@salutejs/plasma-b2c
,@salutejs/plasma-core
,@salutejs/plasma-hope
,@salutejs/plasma-new-hope
,@salutejs/plasma-ui
,@salutejs/plasma-web
@salutejs/plasma-asdk
@salutejs/plasma-tokens
,@salutejs/plasma-typo
@salutejs/plasma-asdk
,@salutejs/plasma-new-hope
@salutejs/plasma-icons
@salutejs/plasma-hope
@salutejs/plasma-b2c
,@salutejs/plasma-hope
,@salutejs/plasma-web
🐛 Bug Fix
.editorconfig
#837 (@Yakutoc)@salutejs/plasma-new-hope
@linaria
deps #845 (@Yakutoc)@salutejs/plasma-web
@salutejs/plasma-hope
@salutejs/plasma-b2c
@salutejs/plasma-cy-utils
,@salutejs/plasma-sb-utils
,@salutejs/plasma-tokens-utils
.editorconfig
#837 (@Yakutoc)@salutejs/plasma-b2c
,@salutejs/plasma-colors
,@salutejs/plasma-core
,@salutejs/plasma-hope
,@salutejs/plasma-icons
,@salutejs/plasma-new-hope
,@salutejs/cra-template-plasma-shop-template
,@salutejs/plasma-temple
,@salutejs/plasma-tokens-b2b
,@salutejs/plasma-tokens-b2c
,@salutejs/plasma-tokens-core
,@salutejs/plasma-tokens-web
,@salutejs/plasma-tokens
,@salutejs/plasma-typo
,@salutejs/plasma-ui
,@salutejs/plasma-web
.editorconfig
#837 (@Yakutoc)@salutejs/plasma-temple
@salutejs/plasma-ui
dev
@salutejs/plasma-asdk
,@salutejs/plasma-b2c
,@salutejs/plasma-hope
,@salutejs/plasma-web
@salutejs/plasma-asdk
,@salutejs/plasma-b2c
,@salutejs/plasma-core
,@salutejs/plasma-hope
,@salutejs/plasma-icons
,@salutejs/plasma-new-hope
,@salutejs/plasma-temple
,@salutejs/plasma-ui
,@salutejs/plasma-web
,@salutejs/plasma-sb-utils
@salutejs/plasma-asdk
,@salutejs/plasma-b2c
,@salutejs/plasma-colors
,@salutejs/plasma-core
,@salutejs/plasma-hope
,@salutejs/plasma-icons
,@salutejs/plasma-new-hope
,@salutejs/cra-template-plasma-shop-template
,@salutejs/plasma-temple
,@salutejs/plasma-tokens-b2b
,@salutejs/plasma-tokens-b2c
,@salutejs/plasma-tokens-core
,@salutejs/plasma-tokens-web
,@salutejs/plasma-tokens
,@salutejs/plasma-typo
,@salutejs/plasma-ui
,@salutejs/plasma-web
,@salutejs/plasma-cy-utils
,@salutejs/plasma-sb-utils
,@salutejs/plasma-tokens-utils
@salutejs/plasma-b2c
,@salutejs/plasma-colors
,@salutejs/plasma-core
,@salutejs/plasma-hope
,@salutejs/plasma-icons
,@salutejs/plasma-new-hope
,@salutejs/cra-template-plasma-shop-template
,@salutejs/plasma-temple
,@salutejs/plasma-tokens-b2b
,@salutejs/plasma-tokens-b2c
,@salutejs/plasma-tokens-core
,@salutejs/plasma-tokens-web
,@salutejs/plasma-tokens
,@salutejs/plasma-typo
,@salutejs/plasma-ui
,@salutejs/plasma-web
,@salutejs/plasma-cy-utils
,@salutejs/plasma-sb-utils
,@salutejs/plasma-tokens-utils
@salutejs/plasma-b2c
,@salutejs/plasma-hope
,@salutejs/plasma-icons
,@salutejs/plasma-temple
,@salutejs/plasma-ui
,@salutejs/plasma-web
@salutejs/plasma-temple
,@salutejs/plasma-ui
@salutejs/plasma-b2c
,@salutejs/plasma-hope
,@salutejs/plasma-new-hope
,@salutejs/plasma-web
Authors: 5