Code review comment for lp:~dholbach/harvest/581719

Revision history for this message
James Westby (james-w) wrote :

Hi,

Does

   <a href="{% url opportunity_edit opportunity.id %}?next={{request.get_full_path}}" target="_blank" class="opportunity-edit-button">{% trans 'edit' %}</a>

have @login_required too?

If so then I'm fine with this change.

Thanks,

James

review: Approve

« Back to merge proposal