EDI Account Creation Pop-ups Don't Match

Bug #1940585 reported by Jennifer Pringle
18
This bug affects 3 people
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.

Revision history for this message
Jennifer Pringle (jpringle-u) wrote :
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Galen Charlton (gmc) wrote :

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
    idlClass="acqedi"
    readonlyFields="id,provider"
    hiddenFields="provider,last_activity"
    [fieldOptions]="{
                      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}
                    }"
    fieldOrder="id,label,host,username,password,account,owner,path,in_dir,vendacct,vendcode,attr_set,use_attrs">
  </eg-fm-record-editor>

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 AddressAlertComponent.

Changed in evergreen:
importance: Undecided → Medium
tags: added: acq-admin acq-edi
removed: edi
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.