added argument labels to help page#122
Conversation
jmmabanta
left a comment
jmmabanta
left a comment
There was a problem hiding this comment.
Looks good! Only needs a slight change in the help UI to make it more intuitive (moving the command list to be inside the parenthesis next to the command instead of num_arguments).
If it wouldn't take to much time, adding the return value labels to the help menu would also be good.
Lastly, can you also update the argument boxes in AutomatedCommands and Flightschedule so that they are labelled correctly (instead of Argument 1, Argument 2, etc.)
| {command.about_info != null ? ( | ||
| <p style={{ paddingLeft: '5%' }}>{command.about_info}</p> | ||
| ) : null} | ||
| {command.arg_labels != null ? ( |
There was a problem hiding this comment.
Can we move the argument list to replace the 'number of arguments' field next to the command name (so it looks like what you would type in).
eg. housekeeping.get_hk(Limit, Before_id, Before_time) instead of housekeeping.get_hk (3)
We can then have this format for return values instead.
2 participants
I'm not sure if we wanted similar thing for what each command returns. Could be displayed in the same sort of way on help page