focalboard icon indicating copy to clipboard operation
focalboard copied to clipboard

Code Debt: Share common code between Person and MultiPerson properties

Open sbishel opened this issue 1 year ago • 4 comments

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.

sbishel avatar Sep 30 '22 15:09 sbishel

May I try this one

cannalee90 avatar Oct 01 '22 13:10 cannalee90

I can work on this issue.

rish-singhal avatar Oct 02 '22 16:10 rish-singhal

@cannalee90 - Thanks assigning to you as first request.

sbishel avatar Oct 03 '22 15:10 sbishel

I'm having struggling with some problems but i think i could do it by next week

cannalee90 avatar Oct 18 '22 14:10 cannalee90