cds-snc / tracker

Check whether a Government of Canada domain is adhering to best security practices.
Other
12 stars 9 forks source link

remove filler column #37

Closed buckley-w-david closed 6 years ago

buckley-w-david commented 6 years ago

This PR removes the code around the temporary "filler" column in the owners list.

This column was there because in Pulse, their CSV had some additional columns that were not relevant to us, so they were removed. However since the row values were accessed by index, that messed up the logic. The filler column was a temporary patch to that until I got around to fixing the access logic.