Skip to content

WIP: EE - Improve slash commands placeholder UI

kushalpandya requested to merge ee-32016-fix-slash-commands-detection into master

What does this MR do?

This MR is an EE port for https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11240

When comment content had slash commands, it tried to strip everything that came after / character which didn't go well when content had anything that involves / (eg; file paths, URLs and etc). This MR uses actual list of supported slash commands for the note and shows appropriate message related to slash command that was present in command. This bug affects the feature introduced in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/10760 which is part of %9.2.

Now user will see following messages while running slash commands.

  • Single Slash Command: Applying command to <command description>.
  • Multiple Slash Commands: Applying multiple commands.

Does this MR meet the acceptance criteria?

What are the relevant issue & merge request numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/32016

https://gitlab.com/gitlab-org/gitlab-ce/issues/27614

Merge request reports

Loading