-
Notifications
You must be signed in to change notification settings - Fork 11.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NEW] Accept quoted slash command arguments #11744
Conversation
Pada Sab, 11 Ogo 2018 1:28 AM Pierre H. Lehnen <notifications@github.com>
menulis:
… This change will make the app engine recognize quoted strings as a single
argument for slash commands, even when they have spaces on it
------------------------------
You can view, comment on, or merge this pull request online at:
#11744
Commit Summary
- Changed the apps api to recognize quotes on slash command arguments
File Changes
- *M* packages/rocketchat-apps/server/bridges/commands.js
<https://github.com/RocketChat/Rocket.Chat/pull/11744/files#diff-0>
(28)
Patch Links:
- https://github.com/RocketChat/Rocket.Chat/pull/11744.patch
- https://github.com/RocketChat/Rocket.Chat/pull/11744.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#11744>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AmmjgoIu_C_hLvXeFM2Tg0xFv6OQMsYQks5uPcLLgaJpZM4V4jTm>
.
|
packages/rocketchat-apps/server/bridges/commands.js
<https://github.com/RocketChat/Rocket.Chat/pull/11744/files#diff-0>
Pada Sab, 11 Ogo 2018 1:31 AM Kap Koi <kapkoi360@gmail.com>menulis:
packages/rocketchat-apps/server/bridges/commands.js
<https://github.com/RocketChat/Rocket.Chat/pull/11744/files#diff-0>
…
Pada Sab, 11 Ogo 2018 1:28 AM Pierre H. Lehnen ***@***.***>
menulis:
> This change will make the app engine recognize quoted strings as a single
> argument for slash commands, even when they have spaces on it
> ------------------------------
> You can view, comment on, or merge this pull request online at:
>
> #11744
> Commit Summary
>
> - Changed the apps api to recognize quotes on slash command arguments
>
> File Changes
>
> - *M* packages/rocketchat-apps/server/bridges/commands.js
> <https://github.com/RocketChat/Rocket.Chat/pull/11744/files#diff-0>
> (28)
>
> Patch Links:
>
> - https://github.com/RocketChat/Rocket.Chat/pull/11744.patch
> - https://github.com/RocketChat/Rocket.Chat/pull/11744.diff
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#11744>, or mute the
> thread
> <https://github.com/notifications/unsubscribe-auth/AmmjgoIu_C_hLvXeFM2Tg0xFv6OQMsYQks5uPcLLgaJpZM4V4jTm>
> .
>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it accepts escaped quotes? like "my \"escaped\" argument"
It does now. |
@Hudell can you fix the lint errors? |
@rodrigok done. |
Could you also resolve #11469 ? |
I'll look into it. |
1ffe613
I tested it and it is working as expected: Input String quoted and multiple lines: |
This change will make the app engine recognize quoted strings as a single argument for slash commands, even when they have spaces on it.
It will also accept multiple lines of arguments
Closes #11469