Bug #16086

always says "There are no inputs" when trying display inputs to a process

Added by Peter Amstutz over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Start date:
01/27/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-


Subtasks

Task #16088: Review 16086-no-inputs-fixResolvedPeter Amstutz

History

#1 Updated by Peter Amstutz over 1 year ago

  • Subject changed from always says "There are no inputs" when trying to display inputs to always says "There are no inputs" when trying display inputs to a process

#2 Updated by Peter Amstutz over 1 year ago

  • Description updated (diff)

#3 Updated by Lucas Di Pentima over 1 year ago

  • Assigned To set to Lucas Di Pentima
  • Status changed from New to In Progress

#4 Updated by Lucas Di Pentima over 1 year ago

Updates at commit 874af8e3 - branch 16086-no-inputs-fix

Fixed an error I made on commit c5fa0c4eed while working on #15407.
Also fixed an issue when displaying start/finish dates.

#5 Updated by Peter Amstutz over 1 year ago

Now when I try to select "inputs" I get a crash page:

TypeError: values.map is not a function
./src/components/chips/chips.tsx/Chips.prototype.render
src/workbench2/src/components/chips/chips.tsx:32

  29 | render() {
  30 |     const { values, filler } = this.props;
  31 |     return <Grid container spacing={8} className={this.props.classes.root}>
> 32 |         {values.map(this.renderChip)}
  33 |         {filler && <Grid item xs>{filler}</Grid>}
  34 |     </Grid>;
  35 | }

#7 Updated by Peter Amstutz over 1 year ago

ce8i5-xvhdp-y7wg0hr7pqo7m3t has the same input shape except the list isn't empty

#8 Updated by Lucas Di Pentima over 1 year ago

Updates at commit cb148df5 ready for review.

#9 Updated by Peter Amstutz over 1 year ago

  • Target version changed from 2020-02-12 Sprint to 2020-01-29 Sprint

#10 Updated by Peter Amstutz over 1 year ago

value: data.mounts[MOUNT_PATH_CWL_INPUT].content[it.id.split('#main/')[1]] || [],

That's not quite right. You should split on '/' and then take the last element.

#11 Updated by Lucas Di Pentima over 1 year ago

Fixed at commit a94a57d2, thanks!

#12 Updated by Peter Amstutz over 1 year ago

Lucas Di Pentima wrote:

Fixed at commit a94a57d2, thanks!

Now that it works I see the existing user interface is a little weird but that's a separate story, this LGTM!

#13 Updated by Lucas Di Pentima over 1 year ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF