if ```row.get(...)``` returns None, then it will set ```identifier``` to
```'None'```, and then later fail to find the case and will generate the
approach error message


On Tue, Apr 15, 2014 at 2:56 PM, Daniel Miller <notifications@github.com>wrote:

> In corehq/apps/data_interfaces/utils.py:
>
> > @@ -16,7 +16,7 @@ def add_cases_to_case_group(domain, case_group_id, uploaded_data):
> >          return response
> >
> >      for row in uploaded_data:
> > -        identifier = row.get('case_identifier')
> > +        identifier = str(row.get('case_identifier'))
>
> What if row.get(...) returns None?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/commcare-hq/pull/3182/files#r11651383>
> .
>