Skip to content

Change registration list from xls to xlsx#61

Closed
mdavis-xyz wants to merge 1 commit into
UNSW-CEEM:masterfrom
mdavis-xyz:xls-x-60
Closed

Change registration list from xls to xlsx#61
mdavis-xyz wants to merge 1 commit into
UNSW-CEEM:masterfrom
mdavis-xyz:xls-x-60

Conversation

@mdavis-xyz
Copy link
Copy Markdown
Contributor

Partly fixes #60

nick-gorman added a commit that referenced this pull request Apr 23, 2026
The cherry-picked commit from mdavis-xyz/NEMOSIS#61 points NEMOSIS at
the .xlsx URL; the local fixture tree mirrors URL paths, so the file
needs the matching extension for the dummy server to serve it.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nick-gorman
Copy link
Copy Markdown
Member

Superseded — the substantive xls→xlsx change (URL + Excel reader) landed via PR #72's ebf27ea, and PR #84 finished the related cleanup (renaming download_xldownload_xlsx and dropping the dead "_downloader.download_xl" dict keys in static_table_url and static_downloader_map). Issue #60 closed by those PRs. Thanks for kicking this off.

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.

Excel file format cannot be determined

2 participants