Fix XADD field value name type

[stfnmllr]

No description provided.

[itamarhaber]

Hello @d024441

Thanks for the PR - I'm unsure what it is supposed to accomplish. A visual inspection of the present docs appears to be ok. Also, the same syntax is used with, for example, HMSET and appears ok.

[stfnmllr]

Hello @itamarhaber

exactly, same syntax is used in HMSET:

{
  "name": ["field", "value"],
  "type": ["string", "string"],
   "multiple": true
} 

but in XADD value and string are in the 'wrong' array:

{
  "name": ["field", "string"],
  "type": ["value", "string"],
  "multiple": true
}

which leads to the following rendered help text in XADD:
XADD key ID field string [field string ...]
which should read
XADD key ID field value [field value ...]

Please be also noticed that the 'Capped streams' option of XADD is completely missing in command.json for XADD.

Best regards

[itamarhaber]

Ok, I see now. Good.

Please be also noticed that the 'Capped streams' option of XADD is completely missing in command.json for XADD.

Thanks for noticing, tracking this at redis/redis-io#167. The problem is that the syntax makes it impossible to express it (i.e. nested optionals).