DiamondLightSource / DiamondLightSource/sci-react-ui

Tweak VisitInput to match "established" Glossary terms

オープン
#93 コメント 6 件 リアクション 0 件 担当者 0 名 GitHub で見る
accepted
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。