data-workspace-frontend icon indicating copy to clipboard operation
data-workspace-frontend copied to clipboard

feat: DT-1834 add option to generate an Id column

Open ian-leggett opened this issue 1 year ago • 0 comments
trafficstars

Description of change

This change enable users to auto generate an ID column when creating new tables from CSV's.

How to test

Before testing both journeys create two CSV files. One that includes an ID column and one that doesn't.

Catalogue page journey

  1. On the catalogue choose a source dataset that you have access to
  2. Within the "Data Tables" section, alongside a table in the list click "Update or restore table"
  3. Click the "Choose file" button and upload the CSV file that doesn't have the ID column and click "Upload CSV"
  4. Scroll to the bottom of the page and where it says "Do you want to generate an ID column?" click "yes" and submit
  5. After the new table has been created go back to the data catalogue page and click on the table you updated
  6. You should now see a new ID column generated in the table

Repeat steps 1 - 6 but this time choose the CSV file with an ID. When you get to step 4 you shouldn't see any option that asks "Do you want to generate an ID column?"

Your files journey

  1. Go to the "Your files" app /files
  2. Upload a CSV file that doesn't have the ID column
  3. Once the file is uploaded click the "Create table" link
  4. On the "Create a table" page just click "Continue"
  5. Choose the schema for your table
  6. Name your table
  7. Scroll to the bottom of the page and where it says "Do you want to generate an ID column?" click "yes" and submit
  8. After the new table has been created view the table by clicking "Open data Explorer" and run the SQL
  9. You should now see a new ID column generated in the table

Repeat steps 1 - 9 but this time choose the CSV file with an ID. When you get to step 7 you shouldn't see any option that asks "Do you want to generate an ID column?"

Checklist

  • [ ] Have tests been added to cover any changes?
  • [ ] Have E2E tests been added to cover any React changes?
  • [ ] Have Accessibility tests been added to cover any React changes?

ian-leggett avatar May 02 '24 13:05 ian-leggett