Add endpoints to set user notification preference - #338
Closed
gwprice wants to merge 3 commits into
Closed
Conversation
Author
The new API uses Django REST Framework. For now, it is designed specifically to support the use cases required by the forum digest notifier (not yet built), with a goal of making it more generally useful over time.
Author
|
This is now stale; closing until the latest code is ready for review |
added 2 commits
July 17, 2013 17:13
Two endpoints are intended to be used by AJAX calls from the web UI with an authenticated user. When enabled, the value of the preference is an encryption of the username, to be used as a token for one-click unsubscribe links. The third endpoint is the target of unsubscribe links, which displays a page with an appropriate message to the user.
|
👍 |
Author
|
@ormsbee Could you look at this one as well? |
Contributor
There was a problem hiding this comment.
Use the require_GET() decorator instead?
Author
There was a problem hiding this comment.
Absolutely. I just didn't know that existed.
Contributor
There was a problem hiding this comment.
FWIW, there's also a similar decorator for @login_required
Author
There was a problem hiding this comment.
I can't use login_required, because that returns a redirect if the requirement is not satisfied. Do we have a library somewhere for general-purpose view decorators?
martynjames
pushed a commit
that referenced
this pull request
Dec 3, 2014
…nges Dcikatic/sol 20 filter changes - merging to get eyes on - still need to confirm correct test coverage
Merged
xavierchan
pushed a commit
to xavierchan/edx-platform-1
that referenced
this pull request
Apr 3, 2019
5640 front end
Danyal-Faheem
pushed a commit
to Danyal-Faheem/edx-platform
that referenced
this pull request
Jul 15, 2025
Signed-off-by: GitHub <noreply@github.com> Co-authored-by: Ali-Salman29 <Ali-Salman29@users.noreply.github.com>
kiram15
pushed a commit
that referenced
this pull request
Jul 17, 2026
…ewStarted feat: decouple enterprise from courseware view redirects
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.
No description provided.