Share one PageNotice across the Trust Policies, Firewall and Previews pages
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 82/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- frontend
Direzione di ricerca
Inizia dai due componenti privati PageNotice in src/features/instance/config/trustPolicies/index.tsx e src/features/instance/config/waf/index.tsx, e da quello condiviso in src/features/instance/config/previews/PageNotice.tsx. Sposta il componente in src/features/instance/config/PageNotice.tsx, poi importalo da tutte e tre le pagine ed elimina le copie. È completato quando le tre pagine renderizzano lo stesso p con le stesse classi e non ci sono altre modifiche al comportamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Three config pages render the same page-level notice with their own copy of one component:
src/features/instance/config/trustPolicies/index.tsx(privatePageNotice)src/features/instance/config/waf/index.tsx(privatePageNotice)src/features/instance/config/previews/PageNotice.tsx, exported and shared within the Previews page, added in #2045
All three are <p className="rounded-md border p-4 text-sm text-muted-foreground">{children}</p>. A change to how these notices look has to be made three times.
Fix: move the component to a module that belongs to the config section, not to one page (for example src/features/instance/config/PageNotice.tsx), and import it from all three pages. Importing the Previews copy from Trust Policies and Firewall would make those pages depend on the Previews feature. There should be no visible change.
Found by the pre-push review of #2045 and deferred from that PR as pre-existing.
Review-Deferral: f1d6177ca3e258b0 severity=nit category=maintainability
- Lingua principale
- TypeScript
- Stelle
- 5
- Fork
- 4
- Merge medio
- 1g 20h
- PR unite (30g)
- 140
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di HarperFast/studio
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
HarperFast/studio#2070 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
HarperFast/studio#2067 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
HarperFast/studio#2026 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
HarperFast/studio#2021 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
HarperFast/studio#1993 ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di HarperFast/studio
Issue simili
-
area:cards enhancement ready-for-agent
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
fil-donadoni/tolaria#8883 ·
I maintainer di solito rispondono entro 1 giorno
-
xsd:integer cast from a string rejects negative numbers, a leading + and surrounding whitespaceApertabug 🐛
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
comunica/comunica#1960 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Waiting for: Product Owner
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
getsentry/sentry-javascript#26203 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
bytedance/flowgram.ai#1235 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
devswha/herdr-web-ui#3325 ·