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に既存コンポーネントを表示する(/sponsorstyles) #847

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

Conversation

jokurebayashi
Copy link
Collaborator

@jokurebayashi jokurebayashi commented Jul 13, 2024

対応Issue

resolve #807

概要

以下のファイルを追加
index.ts
DeleteModal.stories.tsx
EditModal.stories.tsx
OpenAddModalButton.stories.tsx
OpenDeleteModalButton.stories.tsx
OpenEditModalButton.stories.tsx
SponsorStyleAddModal.stories.tsx

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

スクリーンショット 2024-07-13 15 09 04

テスト項目

  • sponsorstylesの各項目が既存コンポーネント通り表示されているか
  • 他に悪影響を与えるエラーがないか

備考

@jokurebayashi jokurebayashi changed the title Storybook Storybookに既存コンポーネントを表示する(/sponsorstyles) Jul 13, 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.

ModalPropsはstorybookで使ってなくて、exportしなくていいかと思いましたのでコメントしました

import { del } from '@api/api_methods';
import { Modal, CloseButton, OutlinePrimaryButton, PrimaryButton } from '@components/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 {

ModalPropsは使っていなので、exportしなくていいかと思います!

@@ -5,7 +5,7 @@ import { put } from '@api/api_methods';
import { PrimaryButton, OutlinePrimaryButton, CloseButton, Input, Modal } from '@components/common';
import { SponsorStyle } 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 {

ここも!

@@ -3,7 +3,7 @@ import React, { useState } from 'react';
import SponsorStyleAddModal from './SponsorStyleAddModal';
import { AddButton } from '@components/common';

interface Props {
export interface Props {
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 Props {
interface Props {

ここも!

@@ -3,7 +3,7 @@ import React, { useState } from 'react';
import DeleteModal from './DeleteModal';
import { DeleteButton } from '@components/common';

interface Props {
export interface Props {
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 Props {
interface Props {

ここも!

Copy link
Collaborator

Choose a reason for hiding this comment

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

ここexportの削除お願いします!

@@ -5,7 +5,7 @@ import { EditButton } from '@components/common';
import EditModal from '@components/sponsorstyles/EditModal';
import { SponsorStyle } from '@type/common';

interface Props {
export interface Props {
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 Props {
interface Props {

Copy link
Collaborator

Choose a reason for hiding this comment

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

ここもです

@@ -5,7 +5,7 @@ import { post } from '@api/api_methods';
import { CloseButton, Input, Modal, PrimaryButton } from '@components/common';
import { SponsorStyle } 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 {

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.

コメントしました〜

@@ -3,7 +3,7 @@ import React, { useState } from 'react';
import DeleteModal from './DeleteModal';
import { DeleteButton } from '@components/common';

interface Props {
export interface Props {
Copy link
Collaborator

Choose a reason for hiding this comment

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

ここexportの削除お願いします!

@@ -5,7 +5,7 @@ import { EditButton } from '@components/common';
import EditModal from '@components/sponsorstyles/EditModal';
import { SponsorStyle } from '@type/common';

interface Props {
export interface Props {
Copy link
Collaborator

Choose a reason for hiding this comment

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

ここもです

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上に表示する(/sponsorstyles)
3 participants