Skip to content

Conversation

dmolesUC
Copy link

@dmolesUC dmolesUC commented Apr 3, 2025

If this pull request addresses an open issue on the repository, please add 'Closes #NN' below, where NN is the issue number.

Closes #384.

Please briefly summarise the changes made in the pull request, and the reason(s) for making these changes.

  • Separates the data dictionary into three separate Markdown tables (one per SQL table), so as not to confuse students when they first import only the surveys table
  • Rearranges fields into the order they appear in the DB Browser for SQL Lite interface (= the order of columns in the CSV, which also becomes the SQL declaration order), so it's easier for students to read down the table as they set data types

If any relevant discussions have taken place elsewhere, please provide links to these.

See #384.

Copy link

github-actions bot commented Apr 3, 2025

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits.

It should be safe to Approve and Run the workflows that need maintainer approval.

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

Successfully merging this pull request may close these issues.

Inclusion of multiple tables in Introduction's data dictionary is potentially confusing
1 participant