Skip to content

Fix SEN resource deployment paths#142

Merged
yqtian-se merged 2 commits into
acmsigsoft:mainfrom
yqtian-se:codex/fix-sen-resource-paths
Jun 16, 2026
Merged

Fix SEN resource deployment paths#142
yqtian-se merged 2 commits into
acmsigsoft:mainfrom
yqtian-se:codex/fix-sen-resource-paths

Conversation

@yqtian-se

@yqtian-se yqtian-se commented Jun 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This PR fixes SEN downloadable resource paths after deployment.

Problem:

  • SEN downloadable files such as ACM_SIGSOFT_pubform.doc were stored under content/SEN/.
  • The site linked to them using uppercase URLs like /SEN/ACM_SIGSOFT_pubform.doc.
  • Hugo generated the files under lower-case paths like /sen/ACM_SIGSOFT_pubform.doc.
  • On case-sensitive deployment hosting, /SEN/... and /sen/... are different paths, so the deployed uppercase URL returned 404.

Changes:

  • Moved SEN downloadable assets from content/SEN/ to static/sen/, so Hugo treats them as stable public files.
  • Updated SEN guideline links to use canonical lower-case /sen/... URLs.
  • Added a deployment copy script to preserve legacy uppercase /SEN/... URLs.
  • Wired the copy script into production and PR-preview workflows.
  • Fixed the Dockerfile Hugo checksum step so the local Docker build works correctly.

@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown

Preview deployed at: https://www2.sigsoft.org/pr-142

@yqtian-se yqtian-se requested a review from Copilot June 15, 2026 09:46
@yqtian-se yqtian-se requested a review from CaptainEmerson June 15, 2026 10:00
Copilot AI reviewed Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@CaptainEmerson CaptainEmerson requested a review from Copilot June 15, 2026 12:48
Copilot AI reviewed Jun 15, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot's findings

  • Files reviewed: 5/12 changed files
  • Comments generated: 1

Comment thread scripts/copy-legacy-sen-assets.mjs
@yqtian-se
Potential fix for pull request finding
c492daa 
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown

Preview deployed at: https://www2.sigsoft.org/pr-142

@yqtian-se yqtian-se merged commit 4f04596 into acmsigsoft:main Jun 16, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@CaptainEmerson CaptainEmerson CaptainEmerson approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yqtian-se @CaptainEmerson