Updated and refactored notifications documentation with new default preferences #1352
Conversation
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
* Product Marketing release notes for Teak
This is a partial update based on a bug report; this file needs deeper review as outlined in the linked ticket.
* chore: documentation * add lti store config image for how-to * updated lti store config documentation and added images * added new page: set up lti consumer, added images * Incorporated suggested changes * Added inline links, removed links from links.txt
The existing documentation on custom certificate templates within an instance is actually enough to make a custom certificate for something other than languages, but doesn't actually make it clear that's what it's doing. This restructures the documentation to make it more obvious that the custom certificate templates work for levels other than language. (For an example of people asking about this, see [this discourse thread](https://discuss.openedx.org/t/trouble-customizing-course-certificates-on-open-edx-tutor-v19/16010) in which people ask for documentation but there's nothing obvious to link them to.)
PR 37331 adds a feature to be able to share to linkedin with an option to consider course level "Organization Name".
* docs: 'how to' view in-context metrics * docs: add Ulmo to release review
This change includes some updates and news screenshots for the notifications feature which will be available in Ulmo release. Added new notification types and improved clarity.
We need to two to work together so put them in one file.
Contributor
|
@ayub02 you need to visit https://github.com/ayub02/docs.openedx.org/tree/main and sync this branch with main, first. Then rebase your branch on your main. |
Contributor
|
Alternatively, I can fix this for you if you can let me know which commits you intended to add in this branch. |
Contributor
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Based on feedback collected from product WG (link), i requested the team to enable default email notifications for 2 notification types:
The PR was merged and backported to Ulmo.
Email notifications for these 2 types were OFF by default because some courses on edx.org had large enrollments and they could lead to higher email cost and could choke the send pipeline.