Problem/Motivation

The admin page allows privileged users to store an API key, then optionally specify a default Brandfolder, then to optionally specify a default Collection within that Brandfolder. There is no asynchronous form submission, so an admin must submit the form three times in order to configure all three of these items. However, the status messages displayed on the admin page do not communicate this flow clearly. There is also the possibility that an API connection error will result in an inaccurate message being displayed to the user.

Steps to reproduce

Visit admin page after installing the module. Note the status message(s) displayed. Note the field values. Submit the form. See attached video.

Proposed resolution

Improve messaging based on current stored config values and API connection success/response.

Remaining tasks

Code updates.

User interface changes

Different messages displayed on admin page based on current state.

API changes

None.

Data model changes

None.

Comments

ndewhurst created an issue. See original summary.

gregory claeyssens’s picture

StatusFileSize
new97.66 KB

I'd also suggest proper messages when an API key is set that is incorrent. The only indication is that the dropdowns below are empty.

Invalid API key

  • ndewhurst committed 53726e6 on 8.x-1.x
    Issue #3170944 by ndewhurst: Config page and sample data improvements....
  • ndewhurst committed b506155 on 8.x-1.x
    Issue #3170944 by ndewhurst: Better messaging on config page.
    
ndewhurst’s picture

Status: Needs work » Fixed

I improved the workflow and messaging here. It is now possible to reset previous Brandfolder/Collection choices. There is form validation to ensure that the given API key grants access to at least one Brandfolder. Finally, if you remove a previously-set API key, the Brandfolder and Collection settings (if any) will be properly cleared out as well.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.