(feat) Add denormalized model fields functionality

apexDev37 commented 3 months ago

Overview

Some models implement a denormalized field (ie. <relation>_count) to store the count value for a related model in the project. Currently, no tracking mechanism (increment or decrement) is implemented for all denormalized fields through one of the following project levels or scopes: endpoints, transactions, or asynchronous tasks.

Intended Outcome

Implement a tracking mechanism (increment or decrement) for all models defining a denormalized field.
Set a default value for fields considered static or with a low modified frequency (ie. constituency_count) (domain knowledge required. Kindly consult if unsure).

IMO, this should preferably be at the transactional level, which adheres to the best practice of having fat models and thin views. This Django paradigm closely maps to a "rich domain model" for those from a DDD world. This can also be done asynchronously, but keep in mind this comes with additional complexity to set up, test, and maintain.

Note

To get started, run the following command to search for all denormalized field definitions:

# assumes you're in the project root.
find . -name 'models.py' -exec grep -H '_count' {} \;

Good ownership means testing your code to communicate the validity of your changes to yourself and other contributors.
In certain contexts, denormalized fields are considered good practice to prevent potential scalability bottlenecks. It's implementation may be overkill here, so anyone who wants to propose a revert to a simpler approach is welcome🙂
Feel free to ask for help from one of the project contributors or me to discuss a solution or pair/ensemble on your idea🌍.
Remember to read the CONTRIBUTING docs after reading this.

C-o-m-o-n commented 3 months ago

Cool. I'll take a look at it.

C-o-m-o-n commented 3 months ago

I have deployed a version of the app btw. It's still behind though. Check it out using this ip. http://13.201.95.101:8000/. I'm still working on it so you can experience some downtime

apexDev37 commented 3 months ago

I have deployed a version of the app btw.

Hey @C-o-m-o-n, appears that the staging link is no longer valid(?)

apexDev37 commented 3 months ago

Cool. I'll take a look at it.

If you haven't already started, I can take this up in my free time.

BenkikoDAO commented 3 months ago

Cool. I'll take a look at it.

If you haven't already started, I can take this up in my free time.

Hello @apexDev37 - Kindly proceed. Sorry for the late recon. @C-o-m-o-n had missed this he is handling another issue.

Friendsofthepeople / django-recall-server