Currently cohort_member is used as a mapping table between patients and cohorts they belong to, with the new columns added to it, it will cease to be just a mapping table. Therefore as @lluismf mentioned we need to introduce a new class named CohortMember that encapsulates the cohort member info, seems to me like that ticket wasn’t well curated to mention what needs to be done