-
Notifications
You must be signed in to change notification settings - Fork 40
Add model for PAF reviewers role, add dashboard view for contracting role #2927
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
Closed
Closed
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
ab4f428
Update Dasboards for finance and contracting
sandeepsajan0 1d62afe
Add model for PAF reviewers roles, and manage permissions for contrac…
sandeepsajan0 d55f9eb
Add PAF status Change form and view with dynamic roles
sandeepsajan0 6c90fa2
Fix Send for Approval view test
sandeepsajan0 1446093
Update users role in approval form tests
sandeepsajan0 09ec9cf
Add request change functionality, send to compliance mail, and added …
sandeepsajan0 a20edbc
fix update status permissions
sandeepsajan0 af7be0f
Add update status form(request change option) for final approver of p…
sandeepsajan0 a9c71a3
Make comments and actions non-editable
sandeepsajan0 0b206a3
Fix existing tests of Final approval form
sandeepsajan0 43455e5
Remove RejectionView from projects
sandeepsajan0 f6aa5cb
Remove RejectionForm and its tests, and Move ProjectApprovalForm from…
sandeepsajan0 470aeb8
Update tests for FinalApproval and PAFStatus Views and forms
sandeepsajan0 9e1eeff
resolve lint issues
sandeepsajan0 7ad3f01
Fix tests for roles by creating PafReviewerRoles via settings
sandeepsajan0 3afeef4
Serve fancybox and search param js file locally
sandeepsajan0 4fa2228
Rebase fixes.
frjo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
34 changes: 34 additions & 0 deletions
34
hypha/apply/dashboard/templates/dashboard/contracting_dashboard.html
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,34 @@ | ||
| {% extends "base-apply.html" %} | ||
| {% load render_table from django_tables2 %} | ||
| {% load i18n static %} | ||
|
|
||
| {% block title %}{% trans "Dashboard" %}{% endblock %} | ||
|
|
||
| {% block content %} | ||
| <div class="admin-bar"> | ||
| <div class="admin-bar__inner admin-bar__inner--with-button"> | ||
| {% block page_header %} | ||
| <h1 class="gamma heading heading--no-margin heading--bold">{% trans "Dashboard" %}</h1> | ||
| {% endblock %} | ||
| <a href="{% url 'wagtailadmin_home' %}" class="button button--primary button--arrow-pixels-white"> | ||
| {% trans "Apply admin" %} | ||
| <svg><use xlink:href="#arrow-head-pixels--solid"></use></svg> | ||
| </a> | ||
| </div> | ||
| </div> | ||
| <div class="wrapper wrapper--large wrapper--inner-space-medium"> | ||
| {% if waiting_for_approval.count %} | ||
| <div id="paf-awaiting-approval" class="wrapper wrapper--bottom-space"> | ||
| <h4 class="heading heading--normal">{% trans "PAF awaiting approval" %}</h4> | ||
| {% render_table waiting_for_approval.table %} | ||
| </div> | ||
| {% endif %} | ||
| </div> | ||
| {% endblock %} | ||
|
|
||
| {% block extra_js %} | ||
| <script src="{% static 'js/apply/url-search-params.js' %}"></script> | ||
| <script src="{% static 'js/apply/submission-filters.js' %}"></script> | ||
| <script src="{% static 'js/apply/submission-tooltips.js' %}"></script> | ||
| <script src="{% static 'js/apply/tabs.js' %}"></script> | ||
| {% endblock %} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
33 changes: 33 additions & 0 deletions
33
hypha/apply/projects/migrations/0054_paf_reviewers_roles__alter_project_status.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| # Generated by Django 3.2.13 on 2022-07-14 12:46 | ||
|
|
||
| from django.db import migrations, models | ||
| import django.db.models.deletion | ||
| import modelcluster.fields | ||
|
|
||
|
|
||
| class Migration(migrations.Migration): | ||
|
|
||
| dependencies = [ | ||
| ('application_projects', '0053_projectapprovalform'), | ||
| ] | ||
|
|
||
| operations = [ | ||
| migrations.AlterField( | ||
| model_name='project', | ||
| name='status', | ||
| field=models.TextField(choices=[('committed', 'Committed'), ('waiting_for_approval', 'Waiting for Approval'), ('contracting', 'Contracting'), ('in_progress', 'In Progress'), ('closing', 'Closing'), ('complete', 'Complete')], default='committed'), | ||
| ), | ||
| migrations.CreateModel( | ||
| name='PAFReviewersRole', | ||
| fields=[ | ||
| ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), | ||
| ('sort_order', models.IntegerField(blank=True, editable=False, null=True)), | ||
| ('role', models.CharField(max_length=200)), | ||
| ('page', modelcluster.fields.ParentalKey(on_delete=django.db.models.deletion.CASCADE, related_name='paf_reviewers_roles', to='application_projects.projectsettings')), | ||
| ], | ||
| options={ | ||
| 'ordering': ['sort_order'], | ||
| 'abstract': False, | ||
| }, | ||
| ), | ||
| ] |
18 changes: 18 additions & 0 deletions
18
hypha/apply/projects/migrations/0055_project_paf_reviews_meta_data.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| # Generated by Django 3.2.13 on 2022-07-18 13:27 | ||
|
|
||
| from django.db import migrations, models | ||
|
|
||
|
|
||
| class Migration(migrations.Migration): | ||
|
|
||
| dependencies = [ | ||
| ('application_projects', '0054_paf_reviewers_roles__alter_project_status'), | ||
| ] | ||
|
|
||
| operations = [ | ||
| migrations.AddField( | ||
| model_name='project', | ||
| name='paf_reviews_meta_data', | ||
| field=models.JSONField(default=dict, help_text='Reviewers role and their actions/comments'), | ||
| ), | ||
| ] |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious why not have these in the
base-apply.htmlif there is no custom javascript code written for the functionalities added to this page. @sandeepsajan0