Skip to content

docs: document fork patches 2 and 3 in PATCHES.md - #4

Merged
Cryptoom merged 1 commit into
mainfrom
fix/mvf-patches-doc-nachtrag
Aug 28, 2026
Merged

Cryptoom merged 1 commit into
mainfrom
fix/mvf-patches-doc-nachtrag

Conversation

@Cryptoom

Copy link
Copy Markdown
Owner

Summary

  • PR feat(design): madevisible brand token reskin for app.css #2 (madevisible brand token reskin, resources/css/app.css) and PR fix(sidebar): remove referral and Discord links #3 (sidebar referral/Discord removal, resources/js/components/AppSidebar.vue) were fork deviations from upstream files, but never recorded in PATCHES.md.
  • Without an entry, a future git merge upstream/main could overwrite these two patches silently, since a conflict-free merge on the same lines just takes the upstream version.
  • Added Patch 2 and Patch 3 entries following the existing Patch 1 format: files touched, marker strings for post-merge verification, and the specific break condition for each.

Test plan

  • Verified no code changes outside PATCHES.md
  • Verified marker strings against actual diffs (git show 32fa724, git show 10ed0ad)
  • Checked for em-dashes/en-dashes in the new text (none)

Patch 2 (madevisible brand token reskin, PR #2) and Patch 3 (sidebar
referral/Discord removal, PR #3) modify upstream files but were not
captured in PATCHES.md. A future upstream merge could silently
overwrite them without conflict. Add marker strings and break
conditions for both, following the existing Patch 1 format.
@Cryptoom
Cryptoom merged commit 6c66b18 into main Aug 28, 2026
@Cryptoom
Cryptoom deleted the fix/mvf-patches-doc-nachtrag branch August 28, 2026 04:46
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.

1 participant