Updated read_google_sheet function to allow for empty columns at the …

[bdgregg]

…end of the Google Sheet rows.

[ojas-uls-dev]

Looks correct. Maybe extract the logic for padding rows into a different function and call it as a function.

Something like

def pad_rows(data):
    ...
    return headers, rows

def read_google_sheet(...):
    ...
    headers, rows = pad_rows(data)
    df = pd.DataFrame(rows, columns = headers)
    return df

[bdgregg]

@ojas-uls-dev, I like what you suggested! I'll see about doing this.

[bdgregg]

@ojas-uls-dev, I believe I have incorporated your suggestion. Can you review my recent change?

[ojas-uls-dev]

That looks correct. Another thing I had a question about was that if there needs to be error checking in case read_google_sheet returns an empty dataframe. Currently, add_columns would just add the required columns into the sheet with default value (None), and the script continues as it would. This is fine if this is intended, but at least a log message might be useful.