LinkinStars commented on PR #1465:
URL: https://github.com/apache/answer/pull/1465#issuecomment-3690845357

   @IfDougelseSa Originally, this notification plugin was designed to provide 
consistent notification functionality when users log in with the same account. 
For example, logging in with Slack would trigger corresponding Slack 
notifications. Of course, the scenario you mentioned is indeed an issue—users 
can configure different login methods and notification channels. 
   
   Therefore, our recommendation is to retain both query methods. By default, 
it will use `externalLogins[0].ExternalID`, but it will also retain the query 
via `fn.Info().SlugName`. This way, if plugins for the same external system 
exist, notifications from that specific external system will take precedence.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to