search

finos / spring-bot

Spring Boot + Java Integration for Symphony/Teams Chat Platform Bots and Apps
https://springbot.finos.org
Apache License 2.0
60 stars 35 forks source link

Fix `User` WorkflowResolver #246

Closed robmoffat closed 2 years ago

robmoffat commented 3 years ago

This is a bug: there are 2 ways of auto-wiring a user:

  1. When a User is mentioned in a message, it gets resolved as a method parameter. (LOW priority)
  2. The sender of the message. (NORMAL priority)

There is a precedence rule here.

However, when there are multiple parameters on the command line, we should 'exhaust' the one that has been used, so that it doesn't fill in all the parameters.