Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/review and secretary pages redesign #688

Merged
merged 24 commits into from
Sep 23, 2024
Merged
Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
3f7e101
feat: turn proposal export list into datatable
EdoStorm96 Aug 22, 2024
b67c5d4
feat:rename stepper block to sidebar
EdoStorm96 Aug 27, 2024
3c5b7fc
fix: change review_detail_sidebar to work with sidebar block on form_…
EdoStorm96 Aug 27, 2024
c15261d
fix: simplify form_buttons
EdoStorm96 Aug 27, 2024
f31e497
feat: restyle review_assign_form
EdoStorm96 Aug 27, 2024
a14cfe1
feat: restyle decision_form
EdoStorm96 Aug 27, 2024
48cc444
feat: restyle review_close_form and some imports
EdoStorm96 Aug 27, 2024
7393982
feat: restyle proposal_confirmation
EdoStorm96 Aug 27, 2024
a0e9323
feat: restyle review_discontinue_form
EdoStorm96 Aug 27, 2024
10a9c2f
feat:restyle proposal_update_date_start
EdoStorm96 Aug 27, 2024
24225c5
feat: restyle & rename proposal_update_dmp
EdoStorm96 Aug 27, 2024
d286713
feat: restyle study_update_attachments
EdoStorm96 Aug 27, 2024
593cebf
fix: update form_buttons block on proposal_submit
EdoStorm96 Aug 27, 2024
b702725
fix: some bugfixes to review_actions logic
EdoStorm96 Aug 27, 2024
c3483cc
feat: restyle committee_members_workload.html
EdoStorm96 Aug 28, 2024
4c29c77
fix: formatting
EdoStorm96 Aug 28, 2024
dee7879
fix: remove redundant with's
EdoStorm96 Sep 9, 2024
04b5b2a
fix: remove lost div
EdoStorm96 Sep 9, 2024
5283ac4
feat: add review sidebar on review close
EdoStorm96 Sep 9, 2024
9c90b5a
fix: updated logic for ChangeAssignment.is_available()
EdoStorm96 Sep 9, 2024
eb39a5d
Merge branch 'major/4' into feature/review_and_secretary_pages_redesign
EdoStorm96 Sep 9, 2024
e52b7e2
style: djlint
EdoStorm96 Sep 9, 2024
887ed5f
Merge branch 'major/4' into feature/review_and_secretary_pages_redesign
EdoStorm96 Sep 17, 2024
97c953a
fix: bugs on fetc_form_base.html
EdoStorm96 Sep 18, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 28 additions & 26 deletions main/templates/base/fetc_form_base.html
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
{# todo: responsive design #}
<div class="col-3">
{% block sidebar %}
{% include "base/stepper.html" %}
{% include stepper %}
{% endblock %}

</div>
Expand All @@ -31,37 +31,39 @@
</div>
{% endblock %}

{{ form }}
{% block post-form-pre-buttons %}{% endblock %}
{% block auto-form-render %}{{ form }}{% endblock %}

<div class="uu-form-text-row">
<div class="uu-form-text">
<div class="mt-4 mb-3 w-100 d-flex">
{% block form-buttons %}
{% include "base/form_buttons.html" %}
{% endblock %}
{% block form-buttons %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think something went wrong when merging. This template now has two {% block form-buttons %} (see below, line 42) which causes an error.

{% block post-form-pre-buttons %}{% endblock %}

</div>
</div>
</div>
{# A block one can use to add stuff after the form, but still next to the stepper #}
{% block post-form %}
<div class="uu-form-text-row">
<div class="uu-form-text">
{% block post-form-text %}{% endblock %}
<div class="mt-4 mb-3 w-100 d-flex">
{% block form-buttons %}
{% include "base/form_buttons.html" %}
{% endblock %}

</div>
</div>
<aside class="uu-form-text-aside">
{% block post-form-aside %}{% endblock %}

</aside>
</div>
{% endblock %}
{# A block one can use to add stuff after the form, but still next to the stepper #}
{% block post-form %}
<div class="uu-form-text-row">
<div class="uu-form-text">
{% block post-form-text %}{% endblock %}

</form>
</div>
{# A block one can use to insert content after the stepper/form container #}
{% block post-form-container-content %}{% endblock %}
</div>
<aside class="uu-form-text-aside">
{% block post-form-aside %}{% endblock %}

</aside>
</div>
{% endblock %}

</div>
{% endblock %}
</form>
</div>
{# A block one can use to insert content after the stepper/form container #}
{% block post-form-container-content %}{% endblock %}

</div>
{% endblock %}
4 changes: 1 addition & 3 deletions reviews/templates/reviews/review_assign_form.html
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,7 @@
{% endblock %}

{% block sidebar %}
{% with review=review %}
{% include "reviews/review_detail_sidebar.html" %}
{% endwith %}
{% include "reviews/review_detail_sidebar.html" %}
{% endblock %}

{% block pre-form-text %}
Expand Down
6 changes: 5 additions & 1 deletion reviews/templates/reviews/review_close_form.html
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
div{% extends "base/fetc_form_base.html" %}
{% extends "base/fetc_form_base.html" %}

{% load static %}
{% load i18n %}
Expand All @@ -7,6 +7,10 @@
{% trans "Review afsluiten" %} - {{ block.super }}
{% endblock %}

{% block sidebar %}
{% include "reviews/review_detail_sidebar.html" %}
{% endblock %}

{% block html_head %}
{{ block.super }}
<script>
Expand Down
6 changes: 1 addition & 5 deletions reviews/templates/reviews/review_detail.html
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,7 @@ <h2>{{ review.proposal.title }} - {{ review.proposal.reference_number }}</h2>
</div>
</div>
<div class="uu-container">
<div class="col-3">
{% with review=review %}
{% include "reviews/review_detail_sidebar.html" %}
{% endwith %}
</div>
<div class="col-3">{% include "reviews/review_detail_sidebar.html" %}</div>
<div class="col-12 col-md-9">
<h4>{% trans "Reviewers" %}</h4>
{% include "reviews/review_table.html" %}
Expand Down
4 changes: 1 addition & 3 deletions reviews/templates/reviews/review_discontinue_form.html
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,7 @@
{% endblock %}

{% block sidebar %}
{% with review=review %}
{% include "reviews/review_detail_sidebar.html" %}
{% endwith %}
{% include "reviews/review_detail_sidebar.html" %}
{% endblock %}

{% block pre-form-text %}
Expand Down
2 changes: 1 addition & 1 deletion reviews/utils/review_actions.py
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ def is_available(self):
return False

if self.review.stage in [
Review.Stages.CLOSING,
Review.Stages.SUPERVISOR,
Review.Stages.CLOSED,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The blame history of this logic is a mess, lol. How did continuations get in here? This should have been caught by someone.

Well, it's been caught now at least. Good find.

In my opinion, this action should be available to the secretary in the following stages:

  • Assignment (obviously)
  • Commission
  • Closing
    • This stage is automatically reached when all opinions have been gathered. But it happens often enough that the secretary wants to add another reviewer before making the final call, so this action should still be available in that stage.

As it stands now, this action will be available in the supervisor stage, which I don't think is right. Could result in weird behaviour if the secretary accidentally adds or removes someone in that stage.

]:
return False
Expand Down
Loading