LesFruitsDefendus / saskatoon-ng

The new generation Saskatoon harverst management system
GNU Affero General Public License v3.0
10 stars 24 forks source link

Organization (beneficiary and/or equipment point) list and detail pages #415

Open oxgr opened 1 month ago

oxgr commented 1 month ago

Type of Issue: Feature request

Beneficiaries should have their own details page to see more information on a particular organisation.

This will especially be beneficial in the future to view more details on equipment available from an organisation.

Expected Behavior:

Current Behavior:

Related URLs:

e.g. https://saskatoon.lesfruitsdefendus.org/beneficiary/

2ynn commented 1 month ago

List pages:

2ynn commented 1 month ago

Detail page:

The Organization detail page should contain:

Adding a + New Equipment button and a AddEquipment form is a nice to have, not a priority. Imo we can deal with it at later point. For now the Organization admin page can be used for that purpose. I don't think equipment lists will change much throughout the season and I expect only a few core members will be in charge of updating this information.

Eventually we will want to add a list of harvests that have reserved this equipment point (or some calendar view?). This is good to keep in mind for layout purposes but this is out of scope for this Issue

oxgr commented 3 weeks ago

Hitting a crossroads in #416

Since beneficiaries and equipment points both will use the same Organization details page and create/update views, should we take the opportunity to rename some things that conflate Organization and Beneficiary? Might save some headache in the future.

e.g. https://github.com/LesFruitsDefendus/saskatoon-ng/blob/3308f376fa89b210b1be4ba02264f411967d0568/saskatoon/member/urls.py#L11-L13

in the same vein, would there be much of a difference in Beneficiaries and Equipment point list pages other than an initial filter? Doing the renaming above might make it simpler to use the same viewset for the two, reducing complexity.

2ynn commented 3 weeks ago

@oxgr I would say let's not change the actual routes but feel free to change the reference name benficiary-create -> organization-create for more clarity

2ynn commented 3 weeks ago

Doing the renaming above might make it simpler to use the same viewset for the two, reducing complexity.

I agree we should not duplicate the viewset and reuse the same for both list pages. Subclassing is fine though if it's needed.

in the same vein, would there be much of a difference in Beneficiaries and Equipment point list pages other than an initial filter?

Some differences I see: