Skip to content

Fix question numbering, remove broken links, and make OneDrive path configurable#282

Merged
Robinlovelace merged 2 commits into263-pre-requisitesfrom
copilot/sub-pr-281
Jan 20, 2026
Merged

Fix question numbering, remove broken links, and make OneDrive path configurable#282
Robinlovelace merged 2 commits into263-pre-requisitesfrom
copilot/sub-pr-281

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026

Addresses review feedback from PR #281 identifying duplicate question numbers, broken navigation links, and hardcoded user paths.

Changes

Precourse questionnaire (precourse.qmd)

  • Fixed duplicate question numbering: Section 1.2 had two Q5s, Section 2 started over at Q4
  • Renumbered all questions sequentially Q1-Q26
  • Removed broken link to non-existent precourse-interactive.html, replaced with note that interactive version is coming

Navigation (_quarto.yml)

  • Commented out link to missing precourse-interactive.qmd file

Build script (scripts/create-docxs.sh)

  • Replaced hardcoded user path with TDS_DEST_DIR environment variable
  • Added null check to prevent failures when variable unset
# Before
DEST_DIR="/c/Users/georl/OneDrive - University of Leeds/career/modules/tds/2025-26"
if [ -d "$DEST_DIR" ]; then

# After
DEST_DIR="${TDS_DEST_DIR:-}"
if [ -n "$DEST_DIR" ] && [ -d "$DEST_DIR" ]; then

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ks, and make path configurable

Co-authored-by: Robinlovelace <1825120+Robinlovelace@users.noreply.github.com>
Copilot AI changed the title [WIP] Add pre-course self-assessment questionnaire for TRAN5340M Fix question numbering, remove broken links, and make OneDrive path configurable Jan 20, 2026
Copilot AI requested a review from Robinlovelace January 20, 2026 17:14
# Copy to OneDrive if it exists
DEST_DIR="/c/Users/georl/OneDrive - University of Leeds/career/modules/tds/2025-26"
if [ -d "$DEST_DIR" ]; then
# Copy to OneDrive if it exists; configure destination via TDS_DEST_DIR environment variable
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Robinlovelace Robinlovelace marked this pull request as ready for review January 20, 2026 18:19
@Robinlovelace Robinlovelace merged commit 7d3cda5 into 263-pre-requisites Jan 20, 2026
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.

2 participants