Skip to content

Adilei readme editing#428

Merged
adilei merged 19 commits into
mainfrom
adilei-readme-review
Dec 14, 2025
Merged

Adilei readme editing#428
adilei merged 19 commits into
mainfrom
adilei-readme-review

Conversation

@adilei
Copy link
Copy Markdown
Collaborator

@adilei adilei commented Dec 14, 2025

No description provided.

Mateusz Wasilewski and others added 19 commits October 31, 2025 17:13
…ve error handling and UI feedback for failed messages
…d from Task to void; remove unused conversation manager field in CopilotStudioAgent
…sation handling; update ChatController and LiveChatService for new conversation flow; enhance frontend to display conversation ID and manage chat state.
…versationManager to support new mapping methods; modify CopilotStudioAgent and MsTeamsProactiveMessage to use ConversationMapping for proactive messaging.
…onfiguration; streamline API endpoints section and remove outdated features.
…AgentUsingSkill

- Reorganize sample under IntegrateWithEngagementHub directory
- Rename folder to HandoverToLiveAgentUsingSkill for clarity
- Update README with improved documentation:
  - Add comprehensive Prerequisites section
  - Clarify skill vs action terminology in mermaid diagram
  - Add detailed explanation of dual app registration requirement
  - Move architecture diagram to 'How It Works' section
  - Add security warnings for appsettings.json
  - Add 'Verify Your Setup' section with testing steps
  - Add comprehensive Troubleshooting section
  - Add Cleanup instructions
  - Replace 'Production Considerations' with 'Future Enhancements'
  - Fix terminology consistency (agent vs bot, skill vs action)
  - Use auto-incrementing numbered lists for easier maintenance
@adilei adilei requested a review from HenryJammes as a code owner December 14, 2025 12:00
@adilei adilei merged commit 743fc10 into main Dec 14, 2025
4 checks passed
@adilei adilei deleted the adilei-readme-review branch December 14, 2025 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant