Ahtisham/add notification requirement#1313
Ahtisham/add notification requirement#1313AhtishamShahid wants to merge 4 commits intooverhangio:ulmofrom
Conversation
|
Hi all - chiming in with the product perspective. We've agreed as the Product WG to make this default installed for the community release. |
Abdul-Muqadim-Arbisoft
moved this from Pending Triage
to In review
in Tutor project management
| tutor-minio@git+https://github.com/overhangio/tutor-minio@main | ||
| tutor-notes@git+https://github.com/overhangio/tutor-notes@main | ||
| tutor-xqueue@git+https://github.com/overhangio/tutor-xqueue@main | ||
| tutor-contrib-notifications@git+https://github.com/openedx/tutor-contrib-notifications.git@main |
There was a problem hiding this comment.
There's a small inconsistency here: all the other plugins in this list are hosted under overhangio/*, but tutor-contrib-notifications is under openedx/*.
Should we align the repository ownership before making this a default plugin? Or is there a reason to keep it under the openedx org?
Also, official plugins don't have *contrib* in their name.
|
I think this PR can be closed because we're planning to make this a default within the platform (as opposed to a Tutor default) per recommendation from @regisb - is my understanding correct, @arbrandes ? |
|
@sarina, yes, that's right. |
7 participants
Description
This PR adds the Notifications plugin as the default offering in tutor