Skip to content

Commit 94557dd

Browse files
committed
Make it clearer for NGOs what fields are missing
1 parent 7b9b8bd commit 94557dd

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

backend/donations/models/ngos.py

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -287,6 +287,9 @@ def mandatory_fields(self):
287287

288288
return [field.field for field in field_names]
289289

290+
def missing_mandatory_fields(self):
291+
return [field for field in self.mandatory_fields if not getattr(self, field.name)]
292+
290293
@property
291294
def mandatory_fields_names(self):
292295
return [field.verbose_name for field in self.mandatory_fields]
@@ -299,6 +302,18 @@ def mandatory_fields_names_lower(self):
299302
def mandatory_fields_names_capitalize(self):
300303
return [field.capitalize() for field in self.mandatory_fields_names]
301304

305+
@property
306+
def missing_mandatory_fields_names(self):
307+
return [field.verbose_name for field in self.missing_mandatory_fields()]
308+
309+
@property
310+
def missing_mandatory_fields_names_lower(self):
311+
return [field.lower() for field in self.missing_mandatory_fields_names]
312+
313+
@property
314+
def missing_mandatory_fields_names_capitalize(self):
315+
return [field.capitalize() for field in self.missing_mandatory_fields_names]
316+
302317
def mandatory_fields_values(self):
303318
return [getattr(self, field.name) for field in self.mandatory_fields]
304319

backend/templates/v2/ngo-account/my-organization/base.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
</p>
1414
{% endblocktranslate %}
1515

16-
{% include "components/info-banner.html" with style="error" banner_message=banner_message banner_list=ngo.mandatory_fields_names_capitalize %}
16+
{% include "components/info-banner.html" with style="error" banner_message=banner_message banner_list=ngo.missing_mandatory_fields_names_capitalize %}
1717
</div>
1818
{% endif %}
1919

0 commit comments

Comments
 (0)