I changed the redirect code and that seems to work ok. Not sure whether we
are talking about another bug now - i fixed the case when the user first
logs in and when the user navigates away from the organisations list page.
It sounds like there are other things going on elsewhere in the system ...

Chris

On Friday, May 24, 2013, Joanna Paulger wrote:

> In
> django/econsensus/publicweb/templates/organizations/organization_list.html:
>
> > @@ -16,7 +16,7 @@
> >  <a title="{% trans 'Add new organization' %}" href="{% url 'organization_add' %}">{% trans "Add" %}</a>
> >  <ul>
> >      {% for organization in organizations %}
> > -    <li><a href="{% url 'publicweb_item_list' organization.slug 'proposal'%}">{{ organization }}</a>
> > +    <li><a href="{% url 'publicweb_item_list' organization.slug 'discussion'%}">{{ organization }}</a>
> >      {% if organization.owner == user %}<span class="org-owner">Owner</span>{% endif %}
>
> Go for it if you fancy fixing the redirecting code:D However, fixing that
> doesn't remove the duplication we have on what the default list status
> should be - plus I just found somewhere else where the proposal type is
> hardcoded as the default, so now we have it in the following places:
>
>    - the template line above
>    - the redirect code
>    - the views.py class DecisionList.set_status (kwargs.get('status',
>    Decision.PROPOSAL_STATUS)) IMO, we should define what the default status
>    should be in only one place in the system, and refer to that one place from
>    the 3 places listed above.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/aptivate/econsensus/pull/122/files#r4381150>
> .
>