DiamondLightSource / DiamondLightSource/sci-react-ui
Tweak VisitInput to match "established" Glossary terms
- Vorherrschende Sprache
- TypeScript
- Sterne
- 8
- Forks
- 3
- Ø Merge
- 3 T. 15 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
"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?
Beitragsleitfaden
Rechercherichtung
Lokalisieren Sie die VisitInput-Komponente und alle ihre Verwendungen und prüfen Sie anschließend, wie ihr Label und ihr Komponentenname derzeit nach außen dargestellt werden. Klären Sie, ob die Änderung nur ein konfigurierbares Label oder zusätzlich eine umbenannte Komponente mit Kompatibilität für bestehende Verwendungen erfordert; done sollte dem etablierten Glossary-Begriff entsprechen, ohne aktuelle VisitInput-Nutzer zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- graphql, typescript
- Bereich
- frontend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 28/100