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.

Sign up for GitHub

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

Jump to bottom

RolePoliciesのコンパネにコピペされている部分を小さなコンポーネントに切り出してもいいかもしれない #14584

Open
KisaragiEffective opened this issue Sep 19, 2024 · 4 comments
Open

RolePoliciesのコンパネにコピペされている部分を小さなコンポーネントに切り出してもいいかもしれない #14584

KisaragiEffective opened this issue Sep 19, 2024 · 4 comments
Labels
packages/frontend Client side specific issue/PR 💚Refactor Rewriting code without changing behavior

Comments

@KisaragiEffective
Copy link
Collaborator

この辺コピペされてるように見えるので小さいコンポーネントに切り出してもいいかもしれない

オリジナルは @KisaragiEffective が #14583 (comment) にポスト
@KisaragiEffective KisaragiEffective added packages/frontend Client side specific issue/PR 💚Refactor Rewriting code without changing behavior labels Sep 19, 2024
@KisaragiEffective KisaragiEffective changed the title RolePoliciesのコンパ燃費コピペされている部分を小さなコンポーネントに切り出してもいいかもしれない RolePoliciesのコンパネにコピペされている部分を小さなコンポーネントに切り出してもいいかもしれない Sep 19, 2024
@kakkokari-gtyih
Copy link
Contributor

型をTypescriptのTypeじゃない方法で記述して引き回す必要がありそう

@KisaragiEffective
Copy link
Collaborator Author

型をTypescriptのTypeじゃない方法で記述して引き回す必要がありそう

i18nのキーの話？

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Sep 19, 2024 •
edited
Loading

ロールポリシーのvalue
（numberなら最大いくつで最小いくつ等も含めて、フォームの構築に必要な値を一元的に持っておくほうが良い）

image

@kakkokari-gtyih
Copy link
Contributor

型をTypescriptのTypeじゃない方法で記述して引き回す必要がありそう

off-topic: この技術は #12952 の実現にも必要

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
packages/frontend Client side specific issue/PR 💚Refactor Rewriting code without changing behavior
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@KisaragiEffective @kakkokari-gtyih