Skip to content

Backbone notifications secondary actions - #253

Merged
singingwolfboy merged 1 commit into
masterfrom
db/notifications-secondary-single
Jun 24, 2013
Merged

Backbone notifications secondary actions#253
singingwolfboy merged 1 commit into
masterfrom
db/notifications-secondary-single

Conversation

@singingwolfboy

Copy link
Copy Markdown
Contributor

Handle secondary actions on notifications either specified as a single object,
or as a list of objects. Under the hood, the initialize method converts a single
object to a list containing a single object.

Handle secondary actions on notifications either specified as a single object,
or as a list of objects. Under the hood, the initialize method converts a single
object to a list containing a single object.
@cahrens

cahrens commented Jun 24, 2013

Copy link
Copy Markdown

👍 Although personally I don't mind the more verbose API (and the clearer expectations and code that go along with it).

@peter-fogg

Copy link
Copy Markdown
Contributor

👍 Looks good.

singingwolfboy added a commit that referenced this pull request Jun 24, 2013
Backbone notifications secondary actions
@singingwolfboy
singingwolfboy merged commit 4d2183d into master Jun 24, 2013
@singingwolfboy
singingwolfboy deleted the db/notifications-secondary-single branch June 24, 2013 18:48
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
antoviaque referenced this pull request in open-craft/openedx-platform Oct 3, 2014
revert deprecated block_type change for local_resource_url
CrewS pushed a commit to CrewS/edx-platform-1 that referenced this pull request Jan 17, 2019
加入我们-把原来那段加回去
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
edx-secure pushed a commit that referenced this pull request Dec 5, 2022
fix: Fix retiring user auth models on disable event
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants