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

既存コンポーンネントをstorybook上で、確認できるようにする(/users) #849

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

jokurebayashi
Copy link
Collaborator

対応Issue

resolve #809

概要

FinanSu/view/next-project/src/components/usersに直接入っている.tsxファイルに対応した.stories.tsxファイルを作成する

画面スクリーンショット等

スクリーンショット 2024-07-18 18 46 41

テスト項目

  • .tsxファイルが.stories.tsxに対応しているか
  • usersの各項目が既存コンポーネント通り表示させているか
  • 他に悪影響のエラーがあるかどうか

備考

なし

…ton.stories.tsx OpenEditModalButton.stories.tsx
@jokurebayashi jokurebayashi changed the title 既存コンポーンネントをstorybook上で、確認できるようにする 既存コンポーンネントをstorybook上で、確認できるようにする(/users) Jul 18, 2024
@jokurebayashi jokurebayashi self-assigned this Jul 18, 2024
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

動作問題なしです
少しコメントしました〜

@@ -4,7 +4,7 @@ import { multiDel } from '@api/api_methods';
import { Modal, CloseButton, OutlinePrimaryButton, PrimaryButton } from '@components/common';
import { User } from '@type/common';

interface ModalProps {
export interface ModalProps {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
export interface ModalProps {
interface ModalProps {

多分exportいらないです!

@@ -6,7 +6,7 @@ import { ROLES } from '@/constants/role';
import { put } from '@api/user';
import { Bureau, User } from '@type/common';

interface ModalProps {
export interface ModalProps {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
export interface ModalProps {
interface ModalProps {

ここも!

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.

既存コンポーネントをStorybook上に表示する(/users)
3 participants