tslittle / tiffanymaster

0 stars 0 forks source link

LP1287309: Create a YAOUS for ordering agency #91

Open tslittle opened 2 years ago

tslittle commented 2 years ago

https://bugs.launchpad.net/evergreen/+bug/1287309

Examples:

Look in config.org_unit_setting_type, and the name is acq.default_copy_location. It says that the data type is "link", and then if you look at how it how is actually created in the seed values file:

'acq.default_copy_location', 'acq', 3131 oils_i18n_gettext('acq.default_copy_location', 3132 'Default copy location', 3133 'coust', 'label'), 3134 oils_i18n_gettext('acq.default_copy_location', 3135 'Default copy location', 3136 'coust', 'description'), 3137 'link', 'acpl')

It's just 'link' and then the fieldmapper code. I wonder if it's as simple as that?

tslittle commented 2 years ago

Narrator: It was not as simple as that.

I saw a bug that I can't find now where Galen said that the YAOUS doesn't understand the array that comes back when it's an org, so it would need to be taught that. And can confirm, I can make the library setting show up, and it's a dropdown of orgs but it doesn't actually apply an id. It also starts messing up the interface.

One interim solution might be to just set the datatype as an integer from the outset. So you'd have to know the id of the org when you're in the UI, which might be janky, but here at PINES we could use it anyway.

tslittle commented 2 years ago

Galen fixed this in his second patch here: https://bugs.launchpad.net/evergreen/+bug/1981714