EDI Account Creation Pop-ups Don't Match
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Confirmed
|
Medium
|
Unassigned |
Bug Description
Evergreen 3.7
New EDI accounts can be created from the EDI Accounts interface (Administration -> Acquisitions Administration -> EDI Accounts) or from within a provider account (Administration -> Acquisitions Administration -> Providers, select your provider and go to the EDI tab).
The pop-up form for creating a EDI account is different depending on whether you go via EDI Accounts or the Provider. The fields are in a different order between the two, the form via Providers has helpful tooltips, and Use EDI Attributes is checked by default on one form but not the other.
The EDI Account interface should use the form from the Providers as the field order is better and it has the tooltips. The form through the Providers doesn't include a field for Provider (since it's being created from within the provider) but that field is needed on the EDI Accounts form.
Changed in evergreen: | |
status: | New → Confirmed |
tags: |
added: acq-admin acq-edi removed: edi |
Noting for whoever works on this that what distinguishes the provider version from the admin page version is not a whole lot:
<eg-fm- record- editor #editDialog "acqedi" lds="id, provider" s="provider, last_activity" ns]="{
host: {helpText: hostHelpStr},
username: {helpText: usernameHelpStr},
password: {helpText: passwordHelpStr},
path: {helpText: pathHelpStr},
in_dir: {helpText: indirHelpStr},
vendacct: {helpText: vendacctHelpStr},
vendcode: {helpText: vendcodeHelpStr},
account: {helpText: accountHelpStr}
} " "id,label, host,username, password, account, owner,path, in_dir, vendacct, vendcode, attr_set, use_attrs" > fm-record- editor>
idlClass=
readonlyFie
hiddenField
[fieldOptio
fieldOrder=
</eg-
If BasicAdminPage were taught hiddenFields as a new @Input and the helpText strings were moved to a shared template somewhere, a new admin interface for EDI Accounts would be nearly as simple as AddressAlertCom ponent.