Conversion errors: 'you may have a duplicate identifier' only specifies one of the sheets the duplicate appears in

[duncandewhurst]

The 'duplicate heading' error message doesn't specify which sheets the duplicate headings are in.

The 'you may have a duplicate identifier' error message only specifies one of the sheets the duplicate appears in.

Including all of the sheet names in the warning would make it easier to troubleshoot errors.

[Bjwebb]

Include sheet names in conversion warnings for duplicate headings
Include sheet names in conversion warnings when an ocid appears on multiple sheets

[robredpath]

Reviewed 2020-08-20:

We think that the "duplicate heading" error might be easier to fix, whereas the second one will be quite a lot harder.

[rhiaro]

@duncandewhurst do you have a sample of the data that caused this error to hand?

[duncandewhurst]

Here you go. In this example, the other sheet which should appear in the error is 'Input: Contracting processes'

[Bjwebb]

We split this into 2 trello cards, 1 for the duplicate header warnings, and 1 for the merge failure.

The first of these has been fixed, and is now deployed to the cove dev site http://cove.dev1.cove.opencontracting.uk0.bigv.io/review/. @duncandewhurst if you get chance, it would be great to know if this works as expected. This should go live over the next couple of days.

The second is still sat on the backlog.

[duncandewhurst]

@duncandewhurst if you get chance, it would be great to know if this works as expected

Looks good!

[Bjwebb]

That fix is live now.

[jpmckinney]

@duncandewhurst Can you update the issue title and description to reflect the remaining (second) issue?