DiamondLightSource / DiamondLightSource/sci-react-ui
Tweak VisitInput to match "established" Glossary terms
- 主要言語
- TypeScript
- スター
- 8
- フォーク
- 3
- 平均マージ
- 3日 15時間
- マージ済み PR(30日)
- 5
説明
"Visit" has been deprecated (in Diamond's GraphQL) for - the more enigmatic - "Instrument Session Id". For reasons.
It should be (at the least) possible to change the label of the current Visit Input to something else, to allow the name change.
However... perhaps the name of the component should be switched over too. In this case it may be better to have two components based on the same underlining code, so as not to break current use of VisitInput - but this is a goodopportunity to refactor the component into more useful parts, usable in future input boxes.
Thoughts?
コントリビューションガイド
調査の方向性
VisitInput コンポーネントとそのすべての使用箇所を特定し、現在そのラベルとコンポーネント名がどのように公開されているかを確認します。この変更に必要なのが設定可能なラベルだけなのか、それとも既存の使用箇所との互換性を備えた名前変更後のコンポーネントも必要なのかを明確にします。done は、現在の VisitInput ユーザーを壊すことなく、確立されている Glossary の用語と一致させる必要があります。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- graphql, typescript
- 領域
- frontend
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 28/100