Skip to content

feat: FormControlのlabel、Fieldsetのlegendにfalsyな値を設定出来ないようruntime validateを追加#5941

Draft
AtsushiM wants to merge 2 commits into
masterfrom
props-validate-for-form-control
Draft

feat: FormControlのlabel、Fieldsetのlegendにfalsyな値を設定出来ないようruntime validateを追加#5941
AtsushiM wants to merge 2 commits into
masterfrom
props-validate-for-form-control

Conversation

@AtsushiM
Copy link
Copy Markdown
Member

関連URL

概要

変更内容

確認方法

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Nov 19, 2025

Open in StackBlitz

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

commit: 0db6efd

@AtsushiM
Copy link
Copy Markdown
Member Author

📝 process.envの設定を調整する必要がある
https://github.com/kufu/smarthr-ui/blob/master/packages/smarthr-ui/rollup.esm.config.js

replace({
values: {
'process.env.NODE_ENV': JSON.stringify('production'),
},
preventAssignment: true,
}),

@Qs-F
Copy link
Copy Markdown
Contributor

Qs-F commented Nov 20, 2025

メモ

- 全環境でprocessが使えるわけではない (そもそもprocess.envはNode.jsのもので、フロントではアクセスできないことも多い)
    - かわりにviteだとimport.metaとかがあるんですが、仕様がちょっと違うんですよねぇ
        - typescriptのトランスパイル時に必ずnodeの処理を通過するはずなんでprocess.envは取得できるような気が…?
            - npm配布時はすでにjsなので…
            - naruhodo
- 今はsmarthr-uiのビルド時にprocess.env.NODE_ENVを `'production'` に置き換える処理を入れているので、これを外さないといけない
- そのため、ちょっとややこしいものを作る必要があり、単純にprocess.env を見に行く、みたいなのだと実は実現できないので、どういう方法が良いかから来期検証していこうとおもってました

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.

2 participants