Arvados

22083-output-collection-missing @ 0646c193d10ecce10518aa815ced1d49ec94a3c0

developer-run-tests: #4519

• All agreed upon points are implemented / addressed. Describe changes from pre-implementation design.
  • yes
• Anything not implemented (discovered or discussed during work) has a follow-up story.
  • n/a
• Code is tested and passing, both automated and manual, what manual testing was done is described.
  • tests running now
• New or changed UX/UX and has gotten feedback from stakeholders.
  • UI changes are minimal, the only new thing is the warning when there's no collection, but this is the same control we use in other places. Otherwise this is a strict improvement, it no longer displays a spinner forever when the output collection is missing.
• Documentation has been updated.
  • n/a
• Behaves appropriately at the intended scale (describe intended scale).
  • no changes to scale
• Considered backwards and forwards compatibility issues between client and server.
  • no changes to API
• Follows our coding standards and GUI style guidelines.
  • yes

Now sets a flag failedToLoadOutputCollection if an exception was thrown while loading the output collection. If this is set, then the UI accordingly either suppresses the collection tab (if the output parameters were available from properties) or renders a warning that the output is not available (if the properties does not include the output parameters).

This lgtm!