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

[Multiplefilter]code recipeへ追加 #362

Merged
merged 5 commits into from
May 26, 2021
Merged

Conversation

penicillin0
Copy link
Contributor

@penicillin0 penicillin0 commented May 26, 2021

やったこと

  • <MultipleFilter />をCodeRecipesへ追加
    • MultipleFilterの背景が透明だったので白に
  • PRのテンプレートをチェックリストに

issue

#340

@netlify
Copy link

netlify bot commented May 26, 2021

✔️ Deploy Preview for ingred-ui ready!

🔨 Explore the source changes: b71cb34

🔍 Inspect the deploy log: https://app.netlify.com/sites/ingred-ui/deploys/60adee87fa184700086932cb

😎 Browse the preview: https://deploy-preview-362--ingred-ui.netlify.app

Comment on lines -395 to +396
<ItemEmpty
title="Title"
subtitle="Sub Title"
imageWidth={100}
imageHeight={100}
/>
<ItemEmpty title="Title" subtitle="Sub Title" imageWidth={100} />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines +3 to +5
## Check List (If️ you added new component in this PR)
- [ ] Export the component in `src/components/index.ts`
- [ ] Add example to `.storybook/documents/Information/Samples/Samples.stories.tsx`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

チェックリストのが気をつけるかなあという気持ちです。

@penicillin0 penicillin0 marked this pull request as ready for review May 26, 2021 06:50
@penicillin0 penicillin0 requested a review from a team as a code owner May 26, 2021 06:50
@penicillin0 penicillin0 requested review from youchann and removed request for a team May 26, 2021 06:50
@penicillin0 penicillin0 changed the title Multiplefilter code recipe [Multiplefilter]code recipeへ追加 May 26, 2021
Copy link
Contributor

@youchann youchann left a comment

Choose a reason for hiding this comment

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

LGTM

@penicillin0 penicillin0 merged commit 0da3465 into master May 26, 2021
@penicillin0 penicillin0 deleted the multiplefilter-code-recipe branch May 26, 2021 07:07
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.

None yet

2 participants