Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accept or Obsolete "Provisional" OEPs #608

Open
sarina opened this issue Jul 5, 2024 · 2 comments
Open

Accept or Obsolete "Provisional" OEPs #608

sarina opened this issue Jul 5, 2024 · 2 comments

Comments

@sarina
Copy link
Contributor

sarina commented Jul 5, 2024

Several OEPs were authored aspirationally, and then never implemented. Other were partially implemented, but have stalled or diverged from reality. Yet others should be followed by the community, but are erroneously marked as "Provisional".

In order to establish OEPs as a trustworthy source of documentation, we should mark OEPs that never came to be as "Replaced" or "Obsolete" (whichever or is more appropriate). We should revise and "Accept" OEPs that are still relevant but mismatched with reality. We should simply "Accept" OEPs that are for all practical purposes adopted by the community. Tagging authors that I think could help give me some insight as to next steps.

  • processes/oep-0055-proc-project-maintainers.rst
  • architectural-decisions/oep-0003-arch-async-tasks.rst
    • @feanil - not sure who to tag here
  • architectural-decisions/oep-0030-arch-pii-markup-and-auditing.rst
  • architectural-decisions/oep-0058-arch-translations-management.rst
    • I think this should be moved to Accepted so I will take it on
  • architectural-decisions/oep-0015-arch-course-wide-js.rst
  • architectural-decisions/oep-0026-arch-realtime-events.rst

Could be adjusted/updated:

  • OEP-45 ADR 2 has the section that begins with "Once this ADR has reached Provisional status:" but the ADR is now accepted, so perhaps this section could be updated?
@bmtcril
Copy link

bmtcril commented Jul 16, 2024

I'll refresh my memory on OEP-30 and try to tackle that first. I think I'd like to get community feedback on how we'd like to proceed with it before taking any action to obsolete it for sure.

OEP-26 should be accepted, but may need a refresh. I'll try to review this week as well.

@msegado
Copy link
Contributor

msegado commented Jul 16, 2024

I believe OEP-15 was implemented by @ha-D in openedx/edx-platform#28411, and the OEP was updated by @xitij2000 in #396 to match the deployed implementation. I'm not sure if things have drifted since then, though?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants