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

feat: AppHeader コンポーネントを追加 #5203

Merged
merged 9 commits into from
Dec 24, 2024
Merged

Conversation

nabeliwo
Copy link
Member

@nabeliwo nabeliwo commented Dec 18, 2024

関連URL

🍐

概要

AppHeader コンポーネントを追加したい。

AppHeader コンポーネントは、Header と AppNavi を結合したコンポーネントです。各プロダクトのヘッダーの表示を共通化させるために存在します。

詳しくは コンポーネントの README を参照してください。

変更内容

AppHeader の追加

確認方法

Storybook で確認してください。
https://63d0ccabb5d2dd29825524ab-vwkuynyozq.chromatic.com/?path=/story/navigation%EF%BC%88%E3%83%8A%E3%83%93%E3%82%B2%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3%EF%BC%89-appheader--playground

Copy link

pkg-pr-new bot commented Dec 18, 2024

Open in Stackblitz

npm i https://pkg.pr.new/kufu/smarthr-ui@5203

commit: 920cb76

@nabeliwo nabeliwo marked this pull request as ready for review December 18, 2024 12:23
@nabeliwo nabeliwo requested a review from a team as a code owner December 18, 2024 12:23
@nabeliwo nabeliwo requested review from yuzuru-akiyama, masa0527, Tokky0425, uknmr and versionfive and removed request for a team, yuzuru-akiyama and masa0527 December 18, 2024 12:23
Copy link
Member

@Tokky0425 Tokky0425 left a comment

Choose a reason for hiding this comment

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

一件コメントしました〜!
プロダクトに入れて動かして見ましたが、コメントしたところ以外は良さそうでした!ありがとうございます! 🙏


import { AppLauncher } from './AppLauncher'
import { Navigation } from './Navigation'
import { UserInfo } from './UserInfo.tsx'
Copy link
Member

Choose a reason for hiding this comment

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

ここの .tsx を消す必要がありそうです! 🙏

Copy link
Member Author

Choose a reason for hiding this comment

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

ほんまですね…!修正しました! 920cb76

Copy link
Member

@Tokky0425 Tokky0425 left a comment

Choose a reason for hiding this comment

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

対応あざます〜 🙏

Copy link
Collaborator

@uknmr uknmr left a comment

Choose a reason for hiding this comment

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

gogo~

@nabeliwo nabeliwo merged commit 4804f44 into master Dec 24, 2024
16 checks passed
@nabeliwo nabeliwo deleted the add-app-header branch December 24, 2024 04:36
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.

3 participants