DiamondLightSource / DiamondLightSource/sci-react-ui

Tweak VisitInput to match "established" Glossary terms

Offen
#93 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
accepted
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.