Commit graph

11 commits

Author SHA1 Message Date
Z.J. van de Weg
c72c76fde3 Fix typos 2016-11-18 14:23:26 +01:00
Z.J. van de Weg
dd826a5f20 Return a consistent not found message
This prevents leakage of project names on an endpoint which is
unauthenticated and thus open to the world.
2016-11-18 12:30:32 +01:00
Z.J. van de Weg
f749fb7fe0 Improve style, add more tests 2016-11-18 11:38:54 +01:00
Z.J. van de Weg
0d04724fa1 More coverage on service level 2016-11-18 10:00:40 +01:00
Z.J. van de Weg
166ee0965b More refactoring, push present to base command 2016-11-17 21:44:26 +01:00
Z.J. van de Weg
1607efa400 Add tests for increased converage 2016-11-17 21:34:24 +01:00
Z.J. van de Weg
6737ada0c8 Remove some commands for now 2016-11-17 21:34:24 +01:00
Z.J. van de Weg
d4def9cbcd Incorporate feedback, improve presenter class
[ci skip]
2016-11-17 21:34:24 +01:00
Z.J. van de Weg
1b4fdb9893 Rename from service, and move to lib/gitlab 2016-11-17 21:34:24 +01:00
Z.J. van de Weg
8c8bc07d32 Refactor and test Slash commands
This is the structure Kamil proposed, which leaves us with a bunch of
smaller classes. This commits deletes outdated files and tests
everything from the SlashCommandService and down (child classes and
subcommands)
2016-11-17 21:34:24 +01:00
Z.J. van de Weg
106b1e39c0 First steps on refactoring Mattermost Slash commands
Now, each subcommand has its own service, plus I've introduced
presenters to be able to delegate the generation of the views.
2016-11-17 21:34:23 +01:00