-
Notifications
You must be signed in to change notification settings - Fork 0
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
Hooks: Events and Filters (Phase 3) #9
Comments
Assigning to myself for better definition. |
@mariajgrmaldi said in the CC meetup on Tuesday that things around events & filters felt pretty finalized. Maria, where are you at with the project? What of the things listed in this ticket are still incomplete, or require tCRIL to be involved? |
@mariajgrmaldi, @felipemontoya: I'm particularly interested in your thoughts around improving documentation for events and filters, as they're going to be two of the primary extension mechanisms we point people to. |
I think that what we have lately refer to as finalized is a state where events and filters where in master, backported to nutmeg (some part that we did not manage to deliver before the cut) and overall usable. That was the initial scope that we set as our goal. Also since this was initially partially funded by a blended project we needed to put the finish line somewhere. Now that this is all done, there is plenty of work to be done to actually make this the primary extension mechanism. I think the most critical areas are:
|
Also we didn't tag @mariajgrimaldi properly 😓 |
Hello! Sorry for not specifying what we've exactly finalized here 😅 Now, I agree that there's much more to do for hooks to be the primary extension strategy. I'm currently trying to improve a pain point for adopters, as Dave said, documentation. This is what I have in mind on doing:
Also, I've been thinking of adding a list of proposed filters/events in each project's repo I created to keep track of our roadmap, but if I'm honest, this hasn't been my priority 😅 |
@feanil - do you have any words of wisdom to impart to @mariajgrimaldi around the documentation needs for events & filters? |
@mariajgrimaldi I think we can use the new root doc site to help unify the documentation. I think guides for how to use openedx events and filters should probably go into the events or filters repo but it sounds like there is also a catalog of events/filters that will be specific to each service and would live in the repo for that service is that right? I'd be happy to help figure out a good organization for this. |
@feanil: yes, that's right. There are a few sections in the docs explaining how to use and test & also some live examples. There are also, as you said, two indexes, one for Open edX Events and one for Open edX Filters. So we could move the indexes to the root docs and link there to each repo documentation. Most code examples in the current doc are related to the LMS, but could we standardize them to be more agnostic to add them to each repository? Or could we use those code examples in the root docs? 🤔 Let me know how we can get started on this. |
We can definitely use the edx-platform examples in the root doc. Here is one example layout I could imagine for docs in openedx-events
Does that help with things might go where? I think the app repos that use openedx-events should just document what events they have and then cross link back to the |
That's great; thanks for the suggestion! That will work for the info we currently have & it will make it easier to add more later. |
There is still some hooks related work in-flight right now, so the timeline to start really hammering out the next phase of this work would be around mid-January. THIS IS VERY MUCH A DRAFT ITEM AT THIS POINT.
That next phase should include at least:
Other items that could go into this phase, but may also be pushed off if we're not ready:
openedx-events
andopenedx-filters
into theopenedx
org (with the current set of eduNEXT developers maintaining commit rights, of course). This may be punted just because we don't yet have an established criterion for when things live in the openedx org vs. elsewhere, but I (@ormsbee) personally support this framework becoming central enough to the platform to justify being in theopenedx
org.The text was updated successfully, but these errors were encountered: