Skip to content

Expect the published app at evilolive.com/Dungeon-Mapper - #198

Merged
evillollive merged 1 commit into
mainfrom
fix/pages-custom-domain
Nov 13, 2026
Merged

evillollive merged 1 commit into
mainfrom
fix/pages-custom-domain

Conversation

@evillollive

Copy link
Copy Markdown
Owner

Summary

The deploy job on main has failed its post-publish check since 42d8dc0 with "Unexpected Pages destination". GitHub Pages now serves the app at evilolive.com/Dungeon-Mapper/ (custom domain on the account), while the check expected evillollive.github.io/Dungeon-Mapper/. The owner chose the evilolive.com address.

Related issues

None filed.

What changed

  • scripts/qualification-artifact.mjs: PAGES_URL is now https://evilolive.com/Dungeon-Mapper/.
  • GitHub currently reports the address as http://. The check accepts that spelling of the same host and path, and nothing else, and always fetches the published files over HTTPS, still with redirects treated as errors and every byte compared to the qualified build.
  • README and docs/SHARING.md links, changelog.

Validation

  • npm run lint
  • qualificationArtifact.test.ts (36 passing, 1 new; the old github.io address and other paths on the new host are rejected)
  • https://evilolive.com/Dungeon-Mapper/ loads the app over HTTPS
  • The real proof is the deploy job on main after merge. It cannot run on a pull request.

Notes for review

  • The check's strictness is unchanged apart from the address. It still fails if the published bytes differ from the tested build.
  • If "Enforce HTTPS" is switched on for the custom domain in GitHub Pages settings, GitHub will report https:// and the http allowance can be removed.

Checklist

  • Tests added/updated where needed
  • Docs updated where needed
  • No secrets introduced

Generated by Claude Code

GitHub Pages now serves the app from the account's custom domain, so the
post-deploy check failed with 'Unexpected Pages destination' on every main
build since 42d8dc0. The check now expects https://evilolive.com/Dungeon-Mapper/,
accepts the http spelling GitHub reports for that same address, and always
reads the published bytes over HTTPS. README and sharing links updated.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VkGH7MqcqEABg9gZkrkZx1
@evillollive
evillollive merged commit 6427824 into main Nov 13, 2026
7 checks passed
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.

3 participants