Skip to content

Commit 4947f78

Browse files
authored
Rename PAF to project form in more places (#4254)
Fixes #4252 Hopefully this is the last of it.
1 parent 2d2687a commit 4947f78

File tree

12 files changed

+1281
-989
lines changed

12 files changed

+1281
-989
lines changed

hypha/apply/activity/adapters/activity_feed.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ def handle_paf_assignment(self, source, paf_approvals, **kwargs):
203203
]
204204
)
205205
users_sentence = " and".join(users.rsplit(",", 1))
206-
return _("PAF assigned to {}").format(users_sentence)
206+
return _("Project form assigned to {}").format(users_sentence)
207207
return None
208208

209209
def handle_task_removal(self, source, task, **kwargs):
Lines changed: 85 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,85 @@
1+
# Generated by Django 4.2.17 on 2024-12-06 07:47
2+
3+
from django.db import migrations, models
4+
5+
6+
class Migration(migrations.Migration):
7+
dependencies = [
8+
("activity", "0084_alter_event_type"),
9+
]
10+
11+
operations = [
12+
migrations.AlterField(
13+
model_name="event",
14+
name="type",
15+
field=models.CharField(
16+
choices=[
17+
("UPDATE_LEAD", "updated lead"),
18+
("BATCH_UPDATE_LEAD", "batch updated lead"),
19+
("EDIT_SUBMISSION", "edited submission"),
20+
("APPLICANT_EDIT", "edited applicant"),
21+
("NEW_SUBMISSION", "submitted new submission"),
22+
("DRAFT_SUBMISSION", "submitted new draft submission"),
23+
("SCREENING", "screened"),
24+
("TRANSITION", "transitioned"),
25+
("BATCH_TRANSITION", "batch transitioned"),
26+
("DETERMINATION_OUTCOME", "sent determination outcome"),
27+
("BATCH_DETERMINATION_OUTCOME", "sent batch determination outcome"),
28+
("INVITED_TO_PROPOSAL", "invited to proposal"),
29+
("REVIEWERS_UPDATED", "updated reviewers"),
30+
("BATCH_REVIEWERS_UPDATED", "batch updated reviewers"),
31+
("PARTNERS_UPDATED", "updated partners"),
32+
("PARTNERS_UPDATED_PARTNER", "partners updated partner"),
33+
("READY_FOR_REVIEW", "marked ready for review"),
34+
("BATCH_READY_FOR_REVIEW", "marked batch ready for review"),
35+
("NEW_REVIEW", "added new review"),
36+
("COMMENT", "added comment"),
37+
("PROPOSAL_SUBMITTED", "submitted proposal"),
38+
("OPENED_SEALED", "opened sealed submission"),
39+
("REVIEW_OPINION", "reviewed opinion"),
40+
("DELETE_SUBMISSION", "deleted submission"),
41+
("DELETE_REVIEW", "deleted review"),
42+
("DELETE_REVIEW_OPINION", "deleted review opinion"),
43+
("CREATED_PROJECT", "created project"),
44+
("UPDATE_PROJECT_LEAD", "updated project lead"),
45+
("UPDATE_PROJECT_TITLE", "updated project title"),
46+
("EDIT_REVIEW", "edited review"),
47+
("SEND_FOR_APPROVAL", "sent for approval"),
48+
("APPROVE_PROJECT", "approved project"),
49+
("ASSIGN_PAF_APPROVER", "assign project form approver"),
50+
("APPROVE_PAF", "approved project form"),
51+
("PROJECT_TRANSITION", "transitioned project"),
52+
("REQUEST_PROJECT_CHANGE", "requested project change"),
53+
("SUBMIT_CONTRACT_DOCUMENTS", "submitted contract documents"),
54+
("UPLOAD_DOCUMENT", "uploaded document to project"),
55+
("REMOVE_DOCUMENT", "removed document from project"),
56+
("UPLOAD_CONTRACT", "uploaded contract to project"),
57+
("APPROVE_CONTRACT", "approved contract"),
58+
("CREATE_INVOICE", "created invoice for project"),
59+
("UPDATE_INVOICE_STATUS", "updated invoice status"),
60+
("APPROVE_INVOICE", "approve invoice"),
61+
("DELETE_INVOICE", "deleted invoice"),
62+
("SENT_TO_COMPLIANCE", "sent project to compliance"),
63+
("UPDATE_INVOICE", "updated invoice"),
64+
("SUBMIT_REPORT", "submitted report"),
65+
("SKIPPED_REPORT", "skipped report"),
66+
("REPORT_FREQUENCY_CHANGED", "changed report frequency"),
67+
("DISABLED_REPORTING", "disabled reporting"),
68+
("REPORT_NOTIFY", "notified report"),
69+
("CREATE_REMINDER", "created reminder"),
70+
("DELETE_REMINDER", "deleted reminder"),
71+
("REVIEW_REMINDER", "reminder to review"),
72+
("BATCH_DELETE_SUBMISSION", "batch deleted submissions"),
73+
("BATCH_ARCHIVE_SUBMISSION", "batch archive submissions"),
74+
("BATCH_INVOICE_STATUS_UPDATE", "batch update invoice status"),
75+
("STAFF_ACCOUNT_CREATED", "created new account"),
76+
("STAFF_ACCOUNT_EDITED", "edited account"),
77+
("ARCHIVE_SUBMISSION", "archived submission"),
78+
("UNARCHIVE_SUBMISSION", "unarchived submission"),
79+
("REMOVE_TASK", "remove task"),
80+
],
81+
max_length=50,
82+
verbose_name="verb",
83+
),
84+
),
85+
]

hypha/apply/activity/options.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,8 @@ class MESSAGES(TextChoices):
4242
EDIT_REVIEW = "EDIT_REVIEW", _("edited review")
4343
SEND_FOR_APPROVAL = "SEND_FOR_APPROVAL", _("sent for approval")
4444
APPROVE_PROJECT = "APPROVE_PROJECT", _("approved project")
45-
ASSIGN_PAF_APPROVER = "ASSIGN_PAF_APPROVER", _("assign paf approver")
46-
APPROVE_PAF = "APPROVE_PAF", _("approved paf")
45+
ASSIGN_PAF_APPROVER = "ASSIGN_PAF_APPROVER", _("assign project form approver")
46+
APPROVE_PAF = "APPROVE_PAF", _("approved project form")
4747
PROJECT_TRANSITION = "PROJECT_TRANSITION", _("transitioned project")
4848
REQUEST_PROJECT_CHANGE = "REQUEST_PROJECT_CHANGE", _("requested project change")
4949
SUBMIT_CONTRACT_DOCUMENTS = (

hypha/apply/activity/templates/messages/email/project_request_change.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
{% block salutation %}{% endblock %}
55

66
{% block content %}{# fmt:off #}
7-
{% trans "A Project has been rejected by PAF reviewers, please update it accordingly and resubmit it to the reviewers." %}
7+
{% trans "A Project has been rejected by project form reviewers, please update it accordingly and resubmit it to the reviewers." %}
88

99
{% trans "Title" %}: {{ source.title }}
1010
{% trans "Link" %}: {{ request.scheme }}://{{ request.get_host }}{% url 'apply:projects:detail' pk=source.pk %}

hypha/apply/dashboard/templates/dashboard/staff_dashboard.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ <h3 class="text-xl mb-2">
9393

9494
{% if PROJECTS_ENABLED and paf_for_review.count %}
9595
<div id="paf_for_review" class="wrapper wrapper--bottom-space">
96-
<h3 class="text-xl mb-2">{% trans "PAFs for review" %}</h3>
96+
<h3 class="text-xl mb-2">{% trans "Project forms for review" %}</h3>
9797
{% render_table paf_for_review.table %}
9898
</div>
9999
{% endif %}

hypha/apply/projects/permissions.py

Lines changed: 36 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -79,13 +79,16 @@ def can_update_paf_approvers(user, project, **kwargs):
7979
return False, "Login Required"
8080

8181
if project.status != INTERNAL_APPROVAL:
82-
return False, "PAF Approvers can be updated only in Internal approval state"
82+
return (
83+
False,
84+
"Project form approvers can be updated only in Internal approval state",
85+
)
8386
if user == project.lead:
8487
return True, "Lead can update approvers in approval state"
8588
if not project.paf_approvals.exists():
8689
return (
8790
False,
88-
"No user can update approvers without paf approval, except lead(lead can add paf approvals)",
91+
"No user can update approvers without project form approval, except lead (lead can add project form approvals)",
8992
)
9093

9194
request = kwargs.get("request")
@@ -99,7 +102,7 @@ def can_update_paf_approvers(user, project, **kwargs):
99102
):
100103
return (
101104
True,
102-
"PAF Reviewer-roles users can update next approval approvers if any approvers assigned",
105+
"Project form reviewer-roles users can update next approval approvers if any approvers assigned",
103106
)
104107
return False, "Forbidden Error"
105108
else:
@@ -114,7 +117,7 @@ def can_update_paf_approvers(user, project, **kwargs):
114117
)
115118
)
116119
if user.id in approvers_ids:
117-
return True, "PAF Reviewer-roles users can update approvers"
120+
return True, "Project form reviewer-roles users can update approvers"
118121
return False, "Forbidden Error"
119122

120123

@@ -126,7 +129,10 @@ def can_update_assigned_paf_approvers(user, project, **kwargs):
126129
if not user.is_authenticated:
127130
return False, "Login Required"
128131
if project.status != INTERNAL_APPROVAL:
129-
return False, "PAF approvers can be assigned only in Internal approval state"
132+
return (
133+
False,
134+
"Project form approvers can be assigned only in Internal approval state",
135+
)
130136
if not project.paf_approvals.exists():
131137
return False, "No user can assign approvers with paf_approvals"
132138

@@ -139,7 +145,7 @@ def can_update_assigned_paf_approvers(user, project, **kwargs):
139145
list(next_paf_approval.paf_reviewer_role.user_roles.all()),
140146
exact_match=True,
141147
):
142-
return True, "PAF Reviewer-roles users can assign approvers"
148+
return True, "Project form reviewer-roles users can assign approvers"
143149
return False, "Forbidden Error"
144150
return False, "Forbidden Error"
145151
else:
@@ -152,7 +158,7 @@ def can_update_assigned_paf_approvers(user, project, **kwargs):
152158
)
153159
)
154160
if user.id in assigners_ids:
155-
return True, "PAF Reviewer-roles users can assign approvers"
161+
return True, "Project form reviewer-roles users can assign approvers"
156162
return False, "Forbidden Error"
157163

158164

@@ -161,9 +167,12 @@ def can_assign_paf_approvers(user, project, **kwargs):
161167
return False, "Login Required"
162168

163169
if project.status != INTERNAL_APPROVAL:
164-
return False, "PAF approvers can be assigned only in Internal approval state"
170+
return (
171+
False,
172+
"Project form approvers can be assigned only in Internal approval state",
173+
)
165174
if not project.paf_approvals.exists():
166-
return False, "No user can assign approvers with paf_approvals"
175+
return False, "No user can assign approvers with project form approvals"
167176

168177
request = kwargs.get("request")
169178
project_settings = ProjectSettings.for_request(request)
@@ -177,7 +186,10 @@ def can_assign_paf_approvers(user, project, **kwargs):
177186
list(next_paf_approval.paf_reviewer_role.user_roles.all()),
178187
exact_match=True,
179188
):
180-
return True, "PAF Reviewer-roles users can assign approvers"
189+
return (
190+
True,
191+
"Project form reviewer-roles users can assign approvers",
192+
)
181193
return False, "Forbidden Error"
182194
return False, "Forbidden Error"
183195
else:
@@ -191,7 +203,7 @@ def can_assign_paf_approvers(user, project, **kwargs):
191203
)
192204

193205
if user.id in assigners_ids:
194-
return True, "PAF Reviewer-roles users can assign approvers"
206+
return True, "Project form reviewer-roles users can assign approvers"
195207
return False, "Forbidden Error"
196208

197209

@@ -200,10 +212,10 @@ def can_update_paf_status(user, project, **kwargs):
200212
return False, "Login Required"
201213

202214
if not project.paf_approvals.filter(approved=False).exists():
203-
return False, "No PAF Approvals Exists"
215+
return False, "No project form approvals exists"
204216

205217
if project.status != INTERNAL_APPROVAL:
206-
return False, "Incorrect project status to approve PAF"
218+
return False, "Incorrect project status to approve project form"
207219

208220
request = kwargs.get("request")
209221
if request:
@@ -219,11 +231,11 @@ def can_update_paf_status(user, project, **kwargs):
219231
if user.id in possible_approvers_ids:
220232
return (
221233
True,
222-
"Next approval group users can approve PAF(For Sequential Approvals)",
234+
"Next approval group users can approve project form (for sequential approvals)",
223235
)
224236
return (
225237
False,
226-
"Only Next approval group can approve PAF(For Sequential Approvals)",
238+
"Only Next approval group can approve project form (for sequential approvals)",
227239
)
228240
else:
229241
possible_approvers_ids = []
@@ -240,11 +252,11 @@ def can_update_paf_status(user, project, **kwargs):
240252
if user.id in possible_approvers_ids:
241253
return (
242254
True,
243-
"All approval group users can approve PAF(For Parallel Approvals)",
255+
"All approval group users can approve project form (for parallel approvals)",
244256
)
245257
return (
246258
False,
247-
"Only approval group users can approve PAF(For Parallel Approvals)",
259+
"Only approval group users can approve project form (for parallel approvals)",
248260
)
249261

250262
return False, "Forbidden Error"
@@ -260,7 +272,10 @@ def can_update_project_status(user, project, **kwargs):
260272
if user.is_apply_staff or user.is_apply_staff_admin:
261273
if project.status == DRAFT:
262274
if no_pafreviewer_role():
263-
return True, "Staff and Staff Admin can skip the PAF approval process"
275+
return (
276+
True,
277+
"Staff and Staff Admin can skip the project form approval process",
278+
)
264279
else:
265280
return True, "Staff and Staff Admin can update status"
266281

@@ -359,7 +374,7 @@ def can_access_project(user, project):
359374
if user.id in paf_reviewer_roles_users_ids:
360375
return (
361376
True,
362-
"PAF Approvers can access the project in Draft, Approval state and after approval state",
377+
"Project form approvers can access the project in Draft, Approval state and after approval state",
363378
)
364379

365380
return False, "Forbidden Error"
@@ -380,9 +395,9 @@ def can_view_contract_category_documents(user, project, **kwargs):
380395

381396
def can_edit_paf(user, project):
382397
if no_pafreviewer_role() and project.status != COMPLETE:
383-
return True, "Paf is editable for active projects if no reviewer roles"
398+
return True, "Project form is editable for active projects if no reviewer roles"
384399
if project.editable_by(user):
385-
return True, "PAF is editable in Draft by this user"
400+
return True, "Project form is editable in Draft by this user"
386401
return False, "You are not allowed to edit the project at this time"
387402

388403

hypha/apply/projects/templates/application_projects/modals/send_for_approval.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,11 @@ <h5>{% trans "Are you ready to submit the project documents to be approved in" %
2727
<p class="flex-grow h-px bg-mid-grey mb-0"></p>
2828
</div>
2929

30-
<p>{% trans "Please note that in "%}{% if project_settings.paf_approval_sequential %}{%trans "sequential order, approvers will approve PAF one after the other."%}{% else %}{% trans "parallel order, approvers can approve PAF anytime." %}{% endif %}</p>
30+
<p>{% trans "Please note that in "%}{% if project_settings.paf_approval_sequential %}{%trans "sequential order, approvers will approve project form one after the other."%}{% else %}{% trans "parallel order, approvers can approve project form anytime." %}{% endif %}</p>
3131
{% url 'apply:projects:submit_project_for_approval' pk=object.id as document_submit_url %}
3232
{% include 'funds/includes/dialog_form_base.html' with form=form value=submit url=document_submit_url %}
3333
{% else %}
34-
<p>{% trans "No PAF Reviewer Roles created yet, please create these in " %}
34+
<p>{% trans "No project form reviewer roles has been created yet, please create these in " %}
3535
<a href="{% project_settings_url project_settings %}" target="_blank">{% trans " project settings" %}</a>
3636
</p>
3737
{% endif %}

hypha/apply/projects/templates/application_projects/modals/update_pafapprovers.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
{% url 'apply:projects:update_pafapprovers' pk=object.id as url %}
99
{% include 'funds/includes/dialog_form_base.html' with form=form value=value url=url %}
1010
{% else %}
11-
<p>{% trans "No PAF Reviewer Roles created yet, please create these in " %}
11+
<p>{% trans "No project form reviewer roles has been created yet, please create these in " %}
1212
<a href="{% project_settings_url project_settings %}" target="_blank">{% trans " project settings" %}</a>
1313
</p>
1414
{% endif %}

hypha/apply/projects/views/project.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1060,13 +1060,15 @@ def post(self, *args, **kwargs):
10601060
break
10611061
else:
10621062
# should never be the case but still to avoid 500.
1063-
raise PermissionDenied("User don't have PAF approver roles")
1063+
raise PermissionDenied(
1064+
"User don't have project form approver roles"
1065+
)
10641066

10651067
paf_status = form.cleaned_data.get("paf_status")
10661068
comment = form.cleaned_data.get("comment", "")
10671069

10681070
paf_status_update_message = _(
1069-
"<p>{role} has updated PAF status to {paf_status}.</p>"
1071+
"<p>{role} has updated project form status to {paf_status}.</p>"
10701072
).format(
10711073
role=paf_approval.paf_reviewer_role.label,
10721074
paf_status=get_paf_status_display(paf_status).lower(),
@@ -1142,7 +1144,7 @@ def post(self, *args, **kwargs):
11421144

11431145
messages.success(
11441146
self.request,
1145-
_("PAF status has been updated"),
1147+
_("Project form status has been updated"),
11461148
extra_tags=PROJECT_ACTION_MESSAGE_TAG,
11471149
)
11481150
elif paf_status == APPROVE:
@@ -1198,7 +1200,7 @@ def post(self, *args, **kwargs):
11981200
)
11991201
messages.success(
12001202
self.request,
1201-
_("PAF has been approved"),
1203+
_("Project form has been approved"),
12021204
extra_tags=PROJECT_ACTION_MESSAGE_TAG,
12031205
)
12041206

@@ -1607,7 +1609,7 @@ def post(self, *args, **kwargs):
16071609

16081610
messages.success(
16091611
self.request,
1610-
_("PAF approvers have been updated"),
1612+
_("Project form approvers have been updated"),
16111613
extra_tags=PROJECT_ACTION_MESSAGE_TAG,
16121614
)
16131615
return HttpResponseClientRefresh()

hypha/apply/users/roles.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ class CommunityReviewer(AbstractUserRole):
7979
class Approver(AbstractUserRole):
8080
role_name = APPROVER_GROUP_NAME
8181
help_text = _(
82-
"Can review/approve PAF, and access compliance documents. "
82+
"Can review/approve project form, and access compliance documents. "
8383
"Must also be in group: Staff, Contracting, or Finance."
8484
)
8585

@@ -89,7 +89,7 @@ class Approver(AbstractUserRole):
8989
class Finance(AbstractUserRole):
9090
role_name = FINANCE_GROUP_NAME
9191
help_text = _(
92-
"Can review/approve the PAF, access documents associated with "
92+
"Can review/approve the project form, access documents associated with "
9393
"contracting, and access invoices approved by Staff."
9494
)
9595

@@ -99,7 +99,7 @@ class Finance(AbstractUserRole):
9999
class Contracting(AbstractUserRole):
100100
role_name = CONTRACTING_GROUP_NAME
101101
help_text = _(
102-
"Can review/approve the PAF and access documents associated with contracting."
102+
"Can review/approve the project form and access documents associated with contracting."
103103
)
104104

105105
available_permissions = {}

0 commit comments

Comments
 (0)