search

csdl / makahiki

An Open Source "Serious Game" Framework for Sustainability
http://makahiki.readthedocs.org/
MIT License
7 stars 5 forks source link

UserNotifications for levels, rounds, quests do not set the content_object. #597

Closed cammoore closed 10 years ago

cammoore commented 10 years ago

Not setting the content object means that there can be multiple SUCCESS or INFO or ERROR UserNotifications. This causes the problem on smartgrid.py 152.

yongwen commented 10 years ago

so far, in UH KC2014, it only affected 3 players. not the highest point earners. The code to display smartgrid should be made robust to catch this error.

On Tue, Jan 28, 2014 at 10:57 AM, Cam Moore notifications@github.comwrote:

Not setting the content object means that there can be multiple SUCCESS or INFO or ERROR UserNotifications. This causes the problem on smartgrid.py 152.

Reply to this email directly or view it on GitHubhttps://github.com/csdl/makahiki/issues/597 .

yongwen commented 10 years ago

fixed and deployed. check for duplicate before creating usernotification object.