Skip to content

Commit dc60811

Browse files
peopleworksclaude
andcommitted
Serve the site's figures from the site
They were linked from raw.githubusercontent.com, so every figure arrived after the page and from a host the site is not published on. Pages now copies docs/assets into the artifact. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent cee5501 commit dc60811

2 files changed

Lines changed: 17 additions & 9 deletions

File tree

.github/workflows/pages.yml

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ on:
55
branches: [main]
66
paths:
77
- 'site/**'
8+
- 'docs/assets/**'
89
- '.github/workflows/pages.yml'
910
workflow_dispatch:
1011

@@ -33,11 +34,18 @@ jobs:
3334
- name: Configure Pages
3435
uses: actions/configure-pages@v5
3536

36-
# The site is a single self-contained HTML file: no build step, no dependencies, nothing
37-
# to go out of date. .nojekyll stops GitHub running Jekyll over it, which would otherwise
38-
# drop any file or directory beginning with an underscore.
37+
# The page itself is one hand-written HTML file: no build step, no dependencies, nothing to
38+
# go out of date. .nojekyll stops GitHub running Jekyll over it, which would otherwise drop
39+
# any file or directory beginning with an underscore.
40+
#
41+
# The figures are copied in rather than linked from raw.githubusercontent.com. Served from
42+
# the same origin they arrive with the page instead of after it, and the site stops
43+
# depending on a host that is not the one it is published on.
3944
- name: Prepare artifact
40-
run: touch site/.nojekyll
45+
run: |
46+
touch site/.nojekyll
47+
mkdir -p site/assets
48+
cp docs/assets/* site/assets/
4149
4250
- name: Upload artifact
4351
uses: actions/upload-pages-artifact@v3

site/index.html

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -889,7 +889,7 @@ <h3>Migrations that ran once</h3>
889889

890890
<figure class="figure">
891891
<div class="fig-frame">
892-
<img src="https://raw.githubusercontent.com/peopleworks/XAFLogicExplainer/main/docs/assets/explainer-editors.png"
892+
<img src="assets/explainer-editors.png"
893893
alt="The custom editors section: a barcode scanner property editor, what it renders, the alias XAF matches on, its base type, and the JavaScript file it depends on — followed by a table of built-in editors reconfigured at run time by a controller."
894894
loading="lazy" style="display:block;width:100%;height:auto">
895895
</div>
@@ -901,7 +901,7 @@ <h3>Migrations that ran once</h3>
901901

902902
<figure class="figure">
903903
<div class="fig-frame">
904-
<img src="https://raw.githubusercontent.com/peopleworks/XAFLogicExplainer/main/docs/assets/explainer-migrations.png"
904+
<img src="assets/explainer-migrations.png"
905905
alt="The migrations section: two version-gated updater blocks, each with the version it upgrades to, the schema phase it ran in, the condition, the methods it calls and the code itself."
906906
loading="lazy" style="display:block;width:100%;height:auto">
907907
</div>
@@ -1052,7 +1052,7 @@ <h2>Most teams have never seen<br>their own domain model.</h2>
10521052

10531053
<figure class="figure">
10541054
<div class="fig-frame">
1055-
<img src="https://raw.githubusercontent.com/peopleworks/XAFLogicExplainer/main/docs/assets/domain-map.gif"
1055+
<img src="assets/domain-map.gif"
10561056
alt="The domain model of a sample XAF application. Hovering an entity dims everything it does not touch, leaving only its own relationships lit."
10571057
loading="lazy" style="display:block;width:100%;height:auto">
10581058
</div>
@@ -1076,7 +1076,7 @@ <h2>Most teams have never seen<br>their own domain model.</h2>
10761076

10771077
<figure class="figure">
10781078
<div class="fig-frame">
1079-
<img src="https://raw.githubusercontent.com/peopleworks/XAFLogicExplainer/main/docs/assets/explainer-entity.png"
1079+
<img src="assets/explainer-entity.png"
10801080
alt="An entity card: every property with its type, which ones are calculated and the expression behind each, and the relationships with the ones whose parent owns the child marked as owned."
10811081
loading="lazy" style="display:block;width:100%;height:auto">
10821082
</div>
@@ -1088,7 +1088,7 @@ <h2>Most teams have never seen<br>their own domain model.</h2>
10881088

10891089
<figure class="figure">
10901090
<div class="fig-frame">
1091-
<img src="https://raw.githubusercontent.com/peopleworks/XAFLogicExplainer/main/docs/assets/explainer-criteria.png"
1091+
<img src="assets/explainer-criteria.png"
10921092
alt="The criteria index: every criteria expression in the application in one table, each with the attribute and entity it came from."
10931093
loading="lazy" style="display:block;width:100%;height:auto">
10941094
</div>

0 commit comments

Comments
 (0)