Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow users to specify index of the header row in sheet config #294

Open
bastienboutonnet opened this issue Dec 20, 2020 · 0 comments
Open

Comments

@bastienboutonnet
Copy link
Owner

bastienboutonnet commented Dec 20, 2020

As seen in #292 and #293 sheets might not have the first row as their header. In that case sheetwork currently fails or when #293 will be implemented it will throw and error that the first row is empty.

In some cases users who are responsible for the ingest may not have rights to edit the sheet or it might not even be handy to do so. In that case we should allow the user to pass a setting in the sheet.yml config that looks something like this:

  - sheet_name: sheet_with_wonky_header
    sheet_key: xxxxxxxxxxxx
    worksheet: sheet1
    header_index: 2 # where the integer corresponds to the row as seen on the GoogleSheet
    target_schema: sand
    target_table: bb_test_sheetwork

NOTE: GoogleSheet indeces do not start from 0 so we need to make sure we handle the conversion under the hood.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant