-
Notifications
You must be signed in to change notification settings - Fork 34
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
Determine process for getting OEP-52 to Accepted #415
Comments
I'm wondering if we could announce this issue with a 1-2 week review period, where we decide that OEP-52 is either accepted, or we nail down what is required to bring it to accepted. |
Update: https://github.com/openedx/open-edx-proposals/pull/414/files has since been merged with no change to the status. We can have another PR to either update the follow-up work with the path to Acceptance or just change the status to Under Review. |
The warning is outdated and can be removed. |
#605 removes the provisional warning |
OEP-52 (Event bus architecture https://open-edx-proposals.readthedocs.io/en/latest/architectural-decisions/oep-0052-arch-event-bus-architecture.html) was initially marked as Provisional in line with the definition of Provisional outlined in OEP-1: https://open-edx-proposals.readthedocs.io/en/latest/processes/oep-0001.html#provisional
There is now a working implementation of the event bus being used in production on edx.org, but it is unclear if this is enough to satisfy the requirement for "viable reference examples and platform adoption" to move the OEP back into Under Review with an eye towards getting it accepted. In particular, it's unclear how far we need to be on the roadmap (openedx/platform-roadmap#28). We have completed the Happy- and Sad-Path epics and are fairly near completion on the Production-ready event.
If we think OEP-52 is already ready to be properly reviewed, I think the proper step would be to update this PR: https://github.com/openedx/open-edx-proposals/pull/414/files to make the status Under Review and add a new acceptance period.
Would appreciate any thoughts on this.
The text was updated successfully, but these errors were encountered: