[SYNPY-1746] Add IsTemplate column to Curator CSV data model

[andrewelamb]

SYNPY-1746

Problem:

The method of using the Parent column in CSV data models was problematic for several reasons.

Solution:

The consensus solution was to add a new column, IsTemplate, that is a boolean like the required column. If the value is True in the IsTemplate column, that row is considered a datatype.

[thomasyu888]

🔥 LGTM! One feedback: from a product perspective, how might one know how to use this column?

I'll defer to @SageGJ for final review - during review please consider how this may impact your generate jsonschema action work and how we advise dcc's to update their data models.

[thomasyu888]

From a Product perspective, I just wanted to confirm that having the "Parent" column will not cause this code to fail. Is this correct?

[andrewelamb]

@thomasyu888 I realized I hadn't updated the documentation, this is fixed now :)

[thomasyu888]

Thanks @andrewelamb, the question remains, will having the Parent column be an issue at all?

[andrewelamb]

Parent is now an optional column. In fact, technically, anything is an optional column. We have it that way so that data modelers can add whatever optional columns they wish, and the Curator Extension will just ignore them.

[SageGJ]

@andrewelamb what's the behavior if both Parent and IsTemplate columns are present?

[andrewelamb]

For creating JSON Schema, the Parent column will now be ignored. (It can still be used for other purposes, like the visualization tool, which was its original intended purpose from what I understand).

[SageGJ]

Got, thanks!