-
Notifications
You must be signed in to change notification settings - Fork 6
[feat] 프론트엔드 주요 기능 구현 및 UI 개선 #398
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
Changes from all commits
a148941
dcce800
910a25e
e271861
b4c1a45
8ecd54b
3386d22
be31346
384047c
cbd4839
47c7f15
baaf58d
408d87d
c27ec66
9fc7eb2
6baf051
81e4d54
efef78a
e4e3b08
77e3788
10a561b
d3ef61c
c5ffa0c
03eb92d
6f223f0
cfa8a91
3d4a74b
edc4bac
f5355f5
48f9820
87d51a8
22851af
ade751b
5ad5eee
bad5d2e
426d7ea
90083ad
ac6b79d
c55ce74
cf779bf
d46d1b0
d367745
fe8ccde
db382bb
a34e013
f5d7a4b
5c687e5
b1684b0
54d68fe
64a7a4d
a1a318b
a4e8927
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| name: 'Chromatic Deployment' | ||
|
|
||
| on: pull_request | ||
|
|
||
| jobs: | ||
| chromatic: | ||
| name: 'Run Chromatic' | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - run: yarn | ||
|
|
||
| - uses: chromaui/action@latest | ||
| id: chromatic | ||
| with: | ||
| projectToken: ${{ secrets.CHROMATIC_PROJECT_TOKEN }} | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: comment PR | ||
| uses: thollander/actions-comment-pull-request@v1 | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| with: | ||
| message: '🚀storybook: ${{ steps.chromatic.outputs.storybookUrl }}' | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,3 +25,9 @@ dist-ssr | |
| .env.local | ||
| # Local Netlify folder | ||
| .netlify | ||
|
|
||
| # TypeScript build cache | ||
| *.tsbuildinfo | ||
|
|
||
|
|
||
| *storybook.log | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| import type { StorybookConfig } from '@storybook/react-vite'; | ||
|
|
||
| const config: StorybookConfig = { | ||
| stories: ['../src/**/*.mdx', '../src/**/*.stories.@(js|jsx|mjs|ts|tsx)'], | ||
| addons: [ | ||
| '@storybook/addon-essentials', | ||
| '@storybook/addon-onboarding', | ||
| '@chromatic-com/storybook', | ||
| ], | ||
| framework: { | ||
| name: '@storybook/react-vite', | ||
| options: {}, | ||
| }, | ||
| }; | ||
| export default config; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| import type { Preview } from '@storybook/react'; | ||
|
|
||
| const preview: Preview = { | ||
| parameters: { | ||
| controls: { | ||
| matchers: { | ||
| color: /(background|color)$/i, | ||
| date: /Date$/i, | ||
| }, | ||
| }, | ||
| }, | ||
| }; | ||
|
|
||
| export default preview; |
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -8,6 +8,10 @@ | |||||||||
| <link rel="mask-icon" href="/src/assets/images/favicon/safari-pinned-tab.svg" color="#0D9488" /> | ||||||||||
| <link rel="manifest" href="/src/assets/images/manifest.json" /> | ||||||||||
| <meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||||||||||
| <meta | ||||||||||
| property="description" | ||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nitpick: 페이지 설명이 표준 권장 수정: 속성을
Suggested change
Original comment in Englishnitpick: The page description is declared with Suggested fix: Change the attribute to
Suggested change
|
||||||||||
| content="트레들리는 체계적이고 효율적인 투자 전략을 제공합니다. 지금 바로 시작해 보세요!" | ||||||||||
| /> | ||||||||||
| <title>트레들리 | 체계적인 투자 전략 플랫폼</title> | ||||||||||
|
|
||||||||||
| <meta property="og:title" content="트레들리 | 체계적인 투자 전략 플랫폼" /> | ||||||||||
|
|
||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚨 issue (security): workflow에서
GITHUB_TOKEN에pull-requests: write권한을 선언하지 않았습니다. 따라서 기본 읽기 전용 토큰을 사용하는 저장소에서는 PR 댓글을 생성하거나 업데이트할 수 없고, Chromatic이 성공하더라도 마지막 단계가 실패합니다.트리거: 저장소의 workflow 토큰이 기본적으로 읽기 전용 권한을 사용할 때 발생합니다.
권장 수정: 최상위에
permissions: pull-requests: write를 선언하거나 필요한 job 수준 권한을 설정하세요.Original comment in English
🚨 issue (security): The workflow does not declare
pull-requests: writepermissions forGITHUB_TOKEN, so repositories using the default read-only token cannot create or update the PR comment and the final step fails despite Chromatic succeeding.Triggers: When the repository's workflow token defaults to read-only permissions.
Suggested fix: Add a top-level
permissions: pull-requests: writedeclaration, or configure the required job-level permission.