Repository navigation
Commit 868b621
Mark text/markdown_text arg_group as not required in chat methods
The [text, markdown_text] mutually_exclusive arg_group in
chat.postEphemeral, chat.postMessage, chat.scheduleMessage and
chat.update was missing a required flag, which downstream consumers
(e.g. slack-ruby-client) interpret as required: true by default,
generating an 'exactly one of' (XOR) validation. In practice these
methods only require at most one of text/markdown_text (they can both
be omitted as long as another required argument, e.g. attachments or
blocks, is present), so add required: false to make this an at-most-one
constraint instead of exactly-one.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Daniel (dB.) Doubrovkine <dblock@dblock.org>1 parent 38cb2f4 commit 868b621
4 files changed
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
151 | 151 | | |
152 | 152 | | |
153 | 153 | | |
| 154 | + | |
154 | 155 | | |
155 | 156 | | |
156 | 157 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
207 | 207 | | |
208 | 208 | | |
209 | 209 | | |
| 210 | + | |
210 | 211 | | |
211 | 212 | | |
212 | 213 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
168 | 168 | | |
169 | 169 | | |
170 | 170 | | |
| 171 | + | |
171 | 172 | | |
172 | 173 | | |
173 | 174 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
183 | 183 | | |
184 | 184 | | |
185 | 185 | | |
| 186 | + | |
186 | 187 | | |
187 | 188 | | |
188 | 189 | | |
| |||
0 commit comments