Groups detail page still says "sequences" after the list page moved to "sightings"

Aperta Adatta ai principianti
  • Qualcosa non va?
#322 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
typescript
Ambito
frontend

Direzione di ricerca

Inizia in frontend/src/pages/SequenceGroupAnnotatePage.tsx, nelle stringhe indicate intorno alle righe 102, 119, 320-328 e 385. Controlla le assertions correlate in frontend/tests/pages/ e aggiorna in modo coerente il copy della pagina di dettaglio, preservando la sequence id reale dove appropriato. Il lavoro è completato quando il flusso dei gruppi usa il vocabolario di sighting concordato e i relativi test del copy passano.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

type: enhancement

#316 renamed the groups list's member count from "Sequences" to "Sightings" and carried the word through that page's copy (subtitle, popover, empty state, column tooltips), on the reasoning that a row is one recurring object and the rest of the app speaks in alerts and objects rather than sequences.

The group detail page was left untouched, so opening a row switches vocabulary mid-flow.

Strings still saying "sequence" in frontend/src/pages/SequenceGroupAnnotatePage.tsx

  • :102 — remove-from-group confirm: Remove sequence #${member.sequence_id} from this group?
  • :119 — member link title: "Open the per-sequence annotation page"
  • :320 — legend: "Label — open any sequence below and label…"
  • :324 — legend: "Validate — confirms every sequence shows…"
  • :328 — legend: "Eject — ✕ removes a sequence that doesn't…"
  • :385 — validate button title: "Confirms every sequence shows the same object and enables label propagation"

Scope note

"Sighting" is a new noun in this app — it exists only on the groups list today. Two coherent endings:

  1. Carry "sighting" into the detail page (this ticket as written), so the groups flow is internally consistent.
  2. Decide "sighting" was the wrong bet and revert the list page to "Sequences" — worse, since "sequence" is an internal data-model word users elsewhere never see.

The member id in the eject confirm is a real sequence id, so that string may want to keep the id while dropping the word (e.g. "Remove this sighting (sequence #123) from the object?"). Worth eyeballing rather than blind-replacing.

No behaviour change; copy only. Tests asserting these strings live in frontend/tests/pages/ (the list-page equivalents were updated in #316).

Lingua principale
TypeScript
Stelle
0
Fork
1
Merge medio
2g 6h
PR unite (30g)
4

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Nessun modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di pyronear/pyro-annotator

Tutte le issue di pyronear/pyro-annotator

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.