IDEMSInternational / rapidpro-flow-toolkit

Toolkit for using spreadsheets to create and modify RapidPro flows
GNU Lesser General Public License v2.1
3 stars 1 forks source link

ContentIndexParser: ignore_row row type #124

Closed geoo89 closed 2 months ago

geoo89 commented 4 months ago

Defines a new content index row type ignore_row. For rows of this type, only the column sheet_name is relevant. Ignores/deletes all previous template/flow/campaign/trigger definitions that match [sheet_name] in the following way:

Template definitions are not affected by ignore_row. The rationale is that often, you use a template to bulk-create a bunch of flows (using data sheets) that are auto-renamed to include the data row ID. You may want to avoid creation of these flows, without deleting the template (in order to create flows using the same template but different data).

Fixes #123

fagiothree commented 4 months ago

I don't think we need a functionality to ignore templates and since I have many cases with template_definition and create_flow sharing the same sheet_name, if I understand correctly if I wanted to use ignore_row to ignore the flow creation this would also delete the template definition?

geoo89 commented 3 months ago

To reword your use case: The rationale is that often, you use a template to bulk-create a bunch of flows (using data sheets) that are auto-renamed to include the data row ID. You may want to avoid creation of these flows, without deleting the template (in order to create flows using the same template but different data). Is that correct?

I amended the PR so that template definitions are not affected. Please have a look.