Skip to content

Commit

Permalink
Add some inline guidance for proposal checking and anonymisation
Browse files Browse the repository at this point in the history
  • Loading branch information
russss committed Feb 7, 2024
1 parent d9840ef commit 25ac41c
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 4 deletions.
2 changes: 1 addition & 1 deletion apps/cfp_review/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,7 @@ def log_and_close(msg, next_page, proposal_id=None):
form.slide_link.data = prop.slide_link

return render_template(
"cfp_review/update_proposal.html", proposal=prop, form=form, next_id=next_id
"cfp_review/proposal.html", proposal=prop, form=form, next_id=next_id
)


Expand Down
3 changes: 2 additions & 1 deletion templates/cfp_review/anonymise_list.html
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{% extends "cfp_review/base.html" %}
{% block title %}Proposals to Anonymise{% endblock %}
{% block body %}
<h2>Proposals to Anonymise<small>(showing {{ proposals | count }})</small></h2>
<h2>Proposals to Anonymise <small>(showing {{ proposals | count }})</small></h2>

<table class="table">
<tr>
Expand Down
17 changes: 15 additions & 2 deletions templates/cfp_review/anonymise_proposal.html
Original file line number Diff line number Diff line change
@@ -1,8 +1,21 @@
{% from "_formhelpers.html" import render_dl_field %}

{% extends "cfp_review/base.html" %}
{% block title %}Anonymise Proposal{% endblock %}
{% block body %}

<h2>Anonymise Proposal</h2>
<div class="well">
<p>Please remove any detail from this proposal which would identify the submitter.</p>
<p>Anonymisation doesn't have to be perfect - if a reviewer knows the topic and the submitter well, they can recuse themselves -
but it should be good enough that someone with a passing knowledge of the topic can't identify the submitter.
</p>
<p>If you're unable to reasonably anonymise this proposal, press "I cannot anonymise this proposal"
and we'll get back to the submitter and work with them to revise it.
</p>
<p>Please don't worry about the content of the proposal. Our reviewers are great - if it's not something we'd
want at EMF, it won't pass review.
</p>
</div>
<form method="post" action="{{ url_for('.anonymise_proposal', proposal_id=proposal.id) }}" class="shrink">
{{ form.hidden_tag() }}
<div class="col-md-8">
Expand All @@ -21,7 +34,7 @@ <h2>{{proposal.title}} <small>by {{proposal.user.name}}</small></h2>
<div class="col-md-4 col-xs-12 btn-group-vertical" role="group">
<div>&nbsp;</div>
<div>&nbsp;</div>
{{ form.anonymise(class_="btn btn-success debounce", tabindex=3) }}
{{ form.anonymise(class_="btn btn-primary debounce", tabindex=3) }}
{{ form.reject(class_="btn btn-danger debounce", tabindex=4) }}
{% if next_proposal %}
<a href="{{ url_for('.anonymise_proposal', proposal_id=next_proposal.id) }}"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,18 @@
{% set active='details' %}
{% include "cfp_review/_propsal_tabs.html" %}


{% if proposal.state == 'new' %}
<div class="well">
<p>You have the ability to mark this proposal as checked, which will send it to anonymisation. Please do so, but only if:</p>
<ul>
<li>You have knowledge of the list of banned speakers and can confirm the submitter is not banned from speaking.</li>
<li>The description is not obviously incomplete.</li>
<li>The description is superficially valid for the category it is submitted in.</li>
</ul>
</div>
{% endif %}

<div class="col-md-12">
<h2>{{proposal.published_title or proposal.title}}<br>
<small>
Expand Down

0 comments on commit 25ac41c

Please sign in to comment.