Share one PageNotice across the Trust Policies, Firewall and Previews pages
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Empieza con los dos componentes privados PageNotice en src/features/instance/config/trustPolicies/index.tsx y src/features/instance/config/waf/index.tsx, y con el compartido en src/features/instance/config/previews/PageNotice.tsx. Mueve el componente a src/features/instance/config/PageNotice.tsx, luego impórtalo desde las tres páginas y elimina las copias. Está terminado cuando las tres páginas renderizan el mismo p con las mismas clases y no hay otros cambios de comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- TypeScript
- Estrellas
- 5
- Forks
- 4
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 140
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de HarperFast/studio
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
HarperFast/studio#2070 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
HarperFast/studio#2067 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
HarperFast/studio#2026 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
HarperFast/studio#2021 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
HarperFast/studio#1993 ·
Los mantenedores suelen responder en 2 días
Todos los issues de HarperFast/studio
Issues similares
-
area:cards enhancement ready-for-agent
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
fil-donadoni/tolaria#8883 ·
Los mantenedores suelen responder en 1 día
-
xsd:integer cast from a string rejects negative numbers, a leading + and surrounding whitespaceAbiertobug 🐛
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
comunica/comunica#1960 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Waiting for: Product Owner
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
getsentry/sentry-javascript#26203 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
bytedance/flowgram.ai#1235 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
devswha/herdr-web-ui#3325 ·