majlis-erc / manuForma

The manuForma application is designed to make TEI data creation and distributed editing faster and easier.
6 stars 1 forks source link

Relations: Add lookup #563

Open mMoliere opened 5 months ago

mMoliere commented 5 months ago

@wsalesky We want to begin work on relations soon. For that to work some modifications need to be made to the lookup feature:

In our schema, the element contains attributes for the kind of relationship (@name) and the persons involved (@active, @passive, @mutual): https://github.com/majlis-erc/manuForma/blob/80f59d1b3e3d071c52c6d1cf9f1d06d8d3e458f4/forms/templates/relations-template.xml#L3

For each of these attributes we want to be able add separate references to person records.

Another complication: the attribute @mutual allows 1+n references.

wsalesky commented 5 months ago

@mMoliere Are you asking for a lookup on each attribute?

mMoliere commented 5 months ago

@wsalesky Yes, exactly. I expect that this will not be a trivial matter.

wsalesky commented 5 months ago

@mMoliere Sounds good. I will see what I can do.

wsalesky commented 5 months ago

@mMoliere Just to be clear, this needs to do lookups across all data types? Persons, places and works?

mMoliere commented 5 months ago

@wsalesky Yes, all data types

mMoliere commented 3 months ago

@wsalesky Please make this feature your highest priority in your work for us.

mMoliere commented 2 months ago

@wsalesky Do you have any news?

wsalesky commented 2 months ago

@mMoliere This is a complicated addition, the forms are not really made for adding multiple values the way you would like to. Unfortunatly I have not had time to experiment with alternative solutions. My summer schedule is very packed, I can bug fixes and small updates, but this will have to wait until the fall. Sorry for the bad news.

mMoliere commented 1 month ago

@wsalesky Ok, but please make this one a priority when you have more time after the summer.

mMoliere commented 3 weeks ago

@wsalesky Have you taken a look at this again?

wsalesky commented 3 weeks ago

Not until September. On Aug 21, 2024, at 7:43 AM, mMoliere @.***> wrote: @wsalesky Have you taken a look at this again?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

mMoliere commented 3 weeks ago

@wsalesky OK, I can work with that. Thanks

wsalesky commented 3 days ago

@mMoliere quick question on this, is @mutual the only attribute that will need to accept multiple values? or should it be enabled for @passive and @active as well?