mattermost-community/focalboard

Code Debt: Share common code between Person and MultiPerson properties

Open

#4842 aperta il 30 set 2022

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)TypeScript (2542 fork)user submission
EnhancementHacktoberfestHelp WantedUp for grabs

Metriche repository

Star
 (26.187 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Summary

The Person Property and MultiPerson Property share most of the same code.

Especially after this PR made them work in an equivalent manner. We should implement a "base Person property" and update Person and MultiPerson to call this property with its specific needs.

How important this is to me and why

Importance: High

As a sameple, see how the various text properties (Text, Email, Phone, and Number) use BaseTextEditor.

Guida contributor

Direzione di ricerca
Esamina i file `webapp/src/properties/person/person.tsx` e `webapp/src/properties/multiperson/multiperson.tsx` per comprenderne la logica condivisa. Rivedi la PR #3888 che li ha resi equivalenti. Crea un componente base simile a `BaseTextEditor` e rifattorizza entrambe le proprietà per estenderlo. Assicurati che i componenti rifattorizzati superino i test esistenti e mantengano la funzionalità.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65