Sure, that's a nice idea.  I'll put that in.

On Sep 25, 2013, at 8:27 AM, Diana Huang <notifications@github.com> wrote:

> In lms/djangoapps/shoppingcart/views.py:
> 
> > @@ -103,12 +110,20 @@ def show_receipt(request, ordernum):
> >      order_items = OrderItem.objects.filter(order=order).select_subclasses()
> >      any_refunds = any(i.status == "refunded" for i in order_items)
> >      receipt_template = 'shoppingcart/receipt.html'
> > +
> > +    # messaging if PaidRegistration was part of order
> > +    any_paid_reg = any(isinstance(i, PaidCourseRegistration) for i in order_items)
> > +    notification = None
> > +    if any_paid_reg:
> > +        notification = (_('Please visit your <a href="{dashboard_link}">dashboard</a> to see your new enrollments.')
> > +                        .format(dashboard_link=reverse('dashboard')))
> I do think it might make sense to do that. I do think it might also make sense to function in OrderItem that gets a list of existing receipt notifications and then returns a new list of notifications. That way, you can check to see if you have the message already in there, and other OrderItems can do their own thing with it.
> 
> —
> Reply to this email directly or view it on GitHub.
>