FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs: restore superseded code-owner monitoring ADR by rahulkanneri-2u · Pull Request #39096 · openedx/openedx-platform · GitHub

docs: restore superseded code-owner monitoring ADR - #39096

Merged
robrap merged 2 commits into
openedx:masterfrom
rahulkanneri-2u:boms-241-restore-superseded-adr
Sep 11, 2026
Merged

robrap merged 2 commits into
openedx:masterfrom
rahulkanneri-2u:boms-241-restore-superseded-adr

Conversation

Copy link
Copy Markdown
Contributor

Description

Restores lms/djangoapps/monitoring/docs/decisions/0001-monitoring-by-code-owner.rst, which was mistakenly deleted (rather than marked superseded) by #39047 as part of the set_code_owner_attribute Celery-decorator cleanup.

Per review feedback on #39047, ADRs documenting a historical decision should be kept and marked Superseded rather than deleted outright, so the record of why the original decision was made isn't lost. This PR restores the file's original content unchanged and updates only its Status field to point at the DEPR issue that supersedes it.

This is a documentation-only change with no functional or runtime impact. It affects Developers/Operators reading architecture decision records, no impact on Learners or Course Authors.

Supporting information

Testing instructions

None required, this is a documentation-only change (one .rst file, no code).

Deadline

None.

Other information

  • No functional or runtime code is touched, this only restores a deleted doc file with its Status field updated to Superseded (DEPR: https://github.com/openedx/edx-django-utils/issues/469). All other content is byte-for-byte identical to the pre-deletion version.
  • No database migration, no security or accessibility implications, no backport needed.
  • Does not depend on any other open PR.

robrap closed this Sep 11, 2026
robrap reopened this Sep 11, 2026
robrap enabled auto-merge (squash) September 11, 2026 15:15
robrap merged commit d1cdd62 into openedx:master Sep 11, 2026
73 of 79 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL