Skip to content

RFC: Ownership of an edit - #48

Open
Maximilian-Nesslauer wants to merge 3 commits into
mainfrom
ownership-of-an-edit
Open

RFC: Ownership of an edit#48
Maximilian-Nesslauer wants to merge 3 commits into
mainfrom
ownership-of-an-edit

Conversation

@Maximilian-Nesslauer

@Maximilian-Nesslauer Maximilian-Nesslauer commented Aug 15, 2026

Copy link
Copy Markdown
Member

RFC 0033 reads the ownership authority out of the document being checked, which for an edit is the one the submitter wrote, and its handover rule says so outright: "verified against the new authority". That is a takeover recorded as the intended behaviour.

This states the rule the check needs instead: an edit verifies against the base version, a change of host verifies against both, and a redirect is not a change of host.

Implemented in KSAModding/content-index#34.

@Maximilian-Nesslauer
Maximilian-Nesslauer requested a review from a team as a code owner August 15, 2026 17:15
@Maximilian-Nesslauer Maximilian-Nesslauer added this to the M1 Core model milestone Aug 15, 2026
@Maximilian-Nesslauer Maximilian-Nesslauer added rfc an RFC proposal or its tracking issue area:index index hosting, distribution, mirrors area:publishing how mod authors get listed stage:proposed RFC is ready for review labels Aug 15, 2026
- RFC 0033 reads the ownership authority out of the document being checked, and for an edit that is the document the submitter wrote
- its handover rule verifies against the new authority, which states the takeover as the intended behaviour
- an edit verifies against the base version, a change of host verifies against both, and a redirect is not a change of host
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:index index hosting, distribution, mirrors area:publishing how mod authors get listed rfc an RFC proposal or its tracking issue stage:proposed RFC is ready for review

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

1 participant