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

plasma-infra(plasma-new-hope): Migrate to storybook 7 [Part 4] #840

Closed
wants to merge 21 commits into from

Conversation

Yakutoc
Copy link
Collaborator

@Yakutoc Yakutoc commented Nov 10, 2023

Release Notes

Переход на Storybook@latest(7.5.3):

What/why Changed

From version 6.5.x to 7.0.0

⚡ Component performance testing

Result: 🟢 OK

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
# or 
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]

@Yakutoc Yakutoc self-assigned this Nov 10, 2023
if (val.true) {
control.control.type = 'boolean';
control.defaultValue = false;
control.table = { defaultValue: { summary: false } };
Copy link
Collaborator Author

@Yakutoc Yakutoc Nov 10, 2023

Choose a reason for hiding this comment

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

[Note]:

https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#no-longer-inferring-default-values-of-args

To manually configure the value that is shown in the ArgsTable doc block, you can configure the table.defaultValue setting:

export default {
  component: MyComponent,
  argTypes: {
    argName: {
      table: { defaultValue: { summary: 'SomeType<T>' } },
    },
  },
};

@Yakutoc Yakutoc force-pushed the feat-migrate-to-storybook-7-new-hope branch 2 times, most recently from 2b25f20 to ccd7b07 Compare November 10, 2023 06:42
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
linaria({
exclude: ['../../../'],
babelOptions: {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

https://github.com/callstack/linaria/blob/master/docs/BUNDLERS_INTEGRATION.md#Vite

If you are using language features that requires a babel transform (such as typescript), ensure the proper babel presets or plugins are passed to linaria.

import { defineConfig } from 'vite';
import linaria from '@linaria/vite';

// example to support typescript syntax:
export default defineConfig(() => ({
  // ...
  plugins: [
    linaria({
      include: ['**/*.{ts,tsx}'],
      babelOptions: {
        presets: ['@babel/preset-typescript', '@babel/preset-react'],
      },
    }),
  ],
}));

@Yakutoc Yakutoc force-pushed the feat-migrate-to-storybook-7-new-hope branch from 7adc3d6 to ccd7b07 Compare November 10, 2023 08:29
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-840/

@Yakutoc Yakutoc force-pushed the feat-migrate-to-storybook-7-new-hope branch from f9eebbe to 07aea04 Compare November 10, 2023 12:03
@Yakutoc Yakutoc force-pushed the feat-migrate-to-storybook-7-new-hope branch from 07aea04 to f83d0dc Compare November 10, 2023 12:29
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
@salute-developers salute-developers deleted a comment from Salute-Eva Nov 10, 2023
@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-840/

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-840/

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-840/

Copy link
Contributor

⚡ Component performance testing

Result: 🟢 OK

@Yakutoc Yakutoc changed the title plasma-new-hope: Migrate to storybook 7 plasma-new-hope: Migrate to storybook 7 [Part 4] Nov 10, 2023
@Yakutoc Yakutoc changed the title plasma-new-hope: Migrate to storybook 7 [Part 4] plasma-infra(plasma-new-hope): Migrate to storybook 7 [Part 4] Nov 10, 2023
@Yakutoc Yakutoc closed this Nov 10, 2023
@Yakutoc Yakutoc deleted the feat-migrate-to-storybook-7-new-hope branch August 22, 2024 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants