Mink icon indicating copy to clipboard operation
Mink copied to clipboard

Current aggregator selected as changed by fallthrough logic is not reflected in the options page

Open machawk1 opened this issue 1 year ago • 4 comments

Largely driven by the changes in #316 via #367, Mink has an option of a series of aggregators for archival sources. If the first one is down, the logic will change to querying the second one.

I am not sure whether the runtime logic persists for subsequent runs but the fall-through, subsequent aggregator is not indicated as the default in the options tab.

This requires some logical work but also UI considerations, e.g.,

  • Should aggregators be sorted or re-arrangeable?
  • Should there be some logic for a random aggregator of those that Mink knows are alive and accessible?

machawk1 avatar Oct 14 '24 17:10 machawk1

The aggregator options are encoded in a <select> in options.html. This aligns with the hard-coded options in content.js, both of which ought to be relocated to some sort of external config file to be referenced in both cases.

machawk1 avatar Oct 14 '24 17:10 machawk1

The fallthrough logic does not appear to work correctly in the main branch but in testing the issue-316 branch (MV3), the logic works (the ODUCS response is malformed) but does not accomplish what this issue (#369) describes. Moving this issue to be a sub-issue of #316, as it ought to be included in the next release, of which #316 is the blocker.

machawk1 avatar Jan 28 '25 21:01 machawk1

Rather, we'll bundle them both under a separate issue (#372)

machawk1 avatar Jan 28 '25 21:01 machawk1

200144f has some logic to set the default aggregator in localStorage when the fallthrough is invoked, but the new default is not shown in the re-ordered options.js page. That dropdown might be hard-coded rather than fetched from localStorage.

Update: It is hardcoded in the options.html. This should be dynamic with a fallback default of the hard-coded values.

machawk1 avatar Jan 28 '25 22:01 machawk1