Skip to content

[Sample] Add time off request card and welcome message middleware#387

Open
rido-min wants to merge 1 commit intonext/corefrom
next/core-teamsbot-sample
Open

[Sample] Add time off request card and welcome message middleware#387
rido-min wants to merge 1 commit intonext/corefrom
next/core-teamsbot-sample

Conversation

@rido-min
Copy link
Member

  • Added a new Adaptive Card for time off requests and updated the "card" handler to use it.
  • Introduced WelcomeMessageMiddleware to send a help message on first interaction; refactored "help" handler to reuse this message.
  • Improved "hello" handler to mention the user and show typing.
  • Commented out the catch-all echo handler to reduce noise.
  • Added OnMessageSubmitAction handler to process Adaptive Card submissions.
  • Minor cleanups: removed verbose logging and improved code structure.

- Added a new Adaptive Card for time off requests and updated the "card" handler to use it.
- Introduced WelcomeMessageMiddleware to send a help message on first interaction; refactored "help" handler to reuse this message.
- Improved "hello" handler to mention the user and show typing.
- Commented out the catch-all echo handler to reduce noise.
- Added OnMessageSubmitAction handler to process Adaptive Card submissions.
- Minor cleanups: removed verbose logging and improved code structure.
string body = activity.ToJson();

logger?.LogTrace("Outgoing Activity :\r {Activity}", body);
//logger?.LogTrace("Outgoing Activity :\r {Activity}", body);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The body is also logged from BotHttpClient

@rido-min rido-min added the CORE label Mar 20, 2026
@rido-min rido-min changed the title Add time off request card and welcome message middleware [Sample] Add time off request card and welcome message middleware Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant