VersionControl:PRs/rabbithole/30 - Revision history

DrivingThatTrain: cryptograss/rabbithole#30, as GitHub has it

2026-11-09T18:06:31Z

cryptograss/rabbithole#30, as GitHub has it

← Older revision		Revision as of 18:06, 9 November 2026
Line 16:		Line 16:
	\|deploy=redeploy pickipedia after merge (with #26).		\|deploy=redeploy pickipedia after merge (with #26).
	}}		}}
			<pre class="pr-body">At 04:35Z on 8 Nov, two saves from the live editor wrote the hardcoded copy of August's chart over Justin's (rev 5132), and Gaven's and Justin's flourishes went with it. Skyler restored it at 04:38Z (rev 15132, the same sha1 as 5132). The save never asked what the page held.

			- `loadTrack` keeps the revision the /Metadata page was read from (`_metadataRevision`).
			- `saveMetadata` checks the page's latest revision first. If it's newer than the one the chart came from, or the page exists when the chart came from no page, nothing is written. The editor says who saved it and when, links the history, and keeps your edits in this browser.
			- The edit also goes with `baserevid` (or `createonly` for a new page), so MediaWiki itself sees the base.
			- After a save, the song's revision moves to the new one, so the next save works.

			Stacked on #26, which makes every song's chart come from the wiki. #26 removes the stale copies; this PR makes sure a save can't overwrite a page that changed since you loaded it. Tests: four new ones for saveMetadata, 163/163 on this branch; build clean. The preview can't save, so a real save needs checking on the live site once this is deployed.

			Deploy: redeploy pickipedia after merge (with #26).
			Worked on by: magent, skyler, justin, in #rabbithole</pre>

DrivingThatTrain: cryptograss/rabbithole#30, as GitHub has it

2026-11-08T05:34:34Z

cryptograss/rabbithole#30, as GitHub has it

← Older revision		Revision as of 05:34, 8 November 2026
Line 5:		Line 5:
	\|url=https://github.com/cryptograss/rabbithole/pull/30		\|url=https://github.com/cryptograss/rabbithole/pull/30
	\|author=magent-cryptograss		\|author=magent-cryptograss
	\|state=~~open~~		\|state=closed
	\|base=main		\|base=main
	\|branch=save-base-revision		\|branch=save-base-revision
Line 11:		Line 11:
	\|pushed=2026-11-08 04:41:53		\|pushed=2026-11-08 04:41:53
	\|merged=		\|merged=
	\|closed=		\|closed=2026-11-08 05:34:31
	\|worked_on_by=magent, skyler, justin		\|worked_on_by=magent, skyler, justin
	\|moods=rabbithole		\|moods=rabbithole
	\|deploy=redeploy pickipedia after merge (with #26).		\|deploy=redeploy pickipedia after merge (with #26).
	}}		}}

DrivingThatTrain: cryptograss/rabbithole#30, as GitHub has it

2026-11-08T04:42:34Z

cryptograss/rabbithole#30, as GitHub has it

New page

{{PR
|repo=rabbithole
|number=30
|title=Save writes over the revision the chart was read from, and no other
|url=https://github.com/cryptograss/rabbithole/pull/30
|author=magent-cryptograss
|state=open
|base=main
|branch=save-base-revision
|opened=2026-11-08 04:42:29
|pushed=2026-11-08 04:41:53
|merged=
|closed=
|worked_on_by=magent, skyler, justin
|moods=rabbithole
|deploy=redeploy pickipedia after merge (with #26).
}}