culturecreates / footlight-app

Footlight CMS Front-end
https://cms.footlight.io
3 stars 0 forks source link

Mandatory settings: Implement Parent term - Child term rule #1348

Open troughc opened 2 months ago

troughc commented 2 months ago

Also see related issue #1309

Context In mandatory settings, most parameters (Fields) are listed by label name. However, some labels are not obviously associated with a particular section. This is the case for Contact parameters.

To make it easier for users to choose what to make mandatory, let's follow the parent term - child term approach we used for event types. This would not affect the display of labels anywhere else.

Action: We will start with Contacts only. Contact fields appear in Events and Organizations.

Additional change request: please reorder the mandatory fields sections to match the left nav (move Place after Event)

Outstanding design question that needs further thought: Currently we are implementing this approach for Contact fields, but it may also be used for Address fields in Places. This needs further thought.

troughc commented 2 months ago

@AbhishekPAnil we can discuss this next week. No action required yet.

troughc commented 2 months ago

@AbhishekPAnil as discussed we will start with Contacts only. We will not change how we handle/dislplay Address fields in the Mandatory fields section. I will add this decision to the description at the top of this issue. Thanks!

troughc commented 1 week ago

@SyamBabu-M you mentioned that you would like to start working on this issue. That is totally fine with me; but I did highlight a couple other issues that I'd like you to see (you should have been sent a couple emails) as they look like quick fixes. But I leave it up to you to determine what makes most sense. See you tomorrow, your evening!

SyamBabu-M commented 2 days ago

Additional change request: please reorder the mandatory fields sections to match the left nav (move Place after Event)

@dev-aravind I believe rearranging the forms array in response of get calendar API will be the best way to approach this. How should we proceed on this?

dev-aravind commented 1 day ago

@SyamBabu-M The forms are now rearranged. Please test this.

SyamBabu-M commented 1 day ago

@dev-aravind Looks good to me.

troughc commented 23 hours ago

Tested: Events section: PASSED Still to do: Organization section - this also has contact fields: in mandatory fields Image

In organiziaitons form Image