Open
Conversation
There was a problem hiding this comment.
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- src/frontend/wwwroot/home/home.html: Language not supported
Comments suppressed due to low confidence (2)
src/backend/agents/baker_agent.py:10
- The function bake_cookies should have a type hint for its return value.
async def bake_cookies(cookie_type: str, quantity: int) -> str:
src/backend/agents/baker_agent.py:13
- The function prepare_dough should have a type hint for its return value.
async def prepare_dough(dough_type: str) -> str:
| agentIcon = "manager"; | ||
| break; | ||
| case "BakerAgent": | ||
| agentIcon = "manager"; |
There was a problem hiding this comment.
The agentIcon for BakerAgent is incorrectly set to 'manager'. It should be set to an icon related to baking.
Suggested change
| agentIcon = "manager"; | |
| agentIcon = 'baker'; |
…g fixes (microsoft#52) * fix: ui changes (#1) * fix: in progress status color after fetch task details (microsoft#5) * cancel notification message updated (microsoft#7) * Update task.js (microsoft#9) * Stages overflow issue fix (microsoft#10) * fix: added space to the agent (microsoft#13) * Approve reject buttons titles disabling buttons and (microsoft#15) * Fix: UX becomes damaged when chat outputs sample code for a task (microsoft#14) * task page UI updates * UI updated code for task * Task page UI updated code * status section UI update in task page * Added custom event (microsoft#24) * feat: added custom event * Logs updated * modify code * added exception logs * added exception logs for cosmos memory --------- Co-authored-by: Roopan P M <v-roopanpm@microsoft.com> * fix: task with zero stages cannot show the page, spins forever and rai test prompt (microsoft#41) * Updated the workflow for build and push docker * updated the repo name * Update requirements.txt * Update requirements.txt * Update requirements.txt * feat: Integrated application insights instrumentation key into the bicep file (microsoft#42) * feat: Integrated application insights instrumentation key into bicep files * added application insights instrumentation key into env and readme file * updated json file * upgraded json file * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * pyLint issues fixed * lint issues fixed --------- Co-authored-by: Roopan-Microsoft <168007406+Roopan-Microsoft@users.noreply.github.com> Co-authored-by: Roopan P M <v-roopanpm@microsoft.com> * Update test.yml (microsoft#43) * Disabling Text Area functionality (microsoft#47) * fix: Usability and Alignments changes (microsoft#48) * Name update and padding removed * fix home page padding and cards height * remove gap in tasks section * Update docker-build-and-push.yml to debug * Update docker-build-and-push.yml * Update docker-build-and-push.yml to take the correct event name * Update docker-build-and-push.yml * fix: text area background color (microsoft#50) * fix: text area background color * Cursor hover style and Light dark color updated --------- Co-authored-by: Prashant-Microsoft <v-pmalusare@microsoft.com> Co-authored-by: Kiran-Siluveru-Microsoft <v-ksiluveru@microsoft.com> Co-authored-by: Mohan-Microsoft <v-mvenudass@microsoft.com> Co-authored-by: Harmanpreet-Microsoft <v-harmanprka@microsoft.com>
…icrosoft#53) * error handled and removed console logs * extra spaces removed
…across various files in the backend (microsoft#58) * fix: ui changes (#1) * fix: in progress status color after fetch task details (microsoft#5) * cancel notification message updated (microsoft#7) * Update task.js (microsoft#9) * Stages overflow issue fix (microsoft#10) * fix: added space to the agent (microsoft#13) * Approve reject buttons titles disabling buttons and (microsoft#15) * Fix: UX becomes damaged when chat outputs sample code for a task (microsoft#14) * task page UI updates * UI updated code for task * Task page UI updated code * status section UI update in task page * Added custom event (microsoft#24) * feat: added custom event * Logs updated * modify code * added exception logs * added exception logs for cosmos memory --------- Co-authored-by: Roopan P M <v-roopanpm@microsoft.com> * fix: task with zero stages cannot show the page, spins forever and rai test prompt (microsoft#41) * Updated the workflow for build and push docker * updated the repo name * Update requirements.txt * Update requirements.txt * Update requirements.txt * feat: Integrated application insights instrumentation key into the bicep file (microsoft#42) * feat: Integrated application insights instrumentation key into bicep files * added application insights instrumentation key into env and readme file * updated json file * upgraded json file * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * Update docker-build-and-push.yml * pyLint issues fixed * lint issues fixed --------- Co-authored-by: Roopan-Microsoft <168007406+Roopan-Microsoft@users.noreply.github.com> Co-authored-by: Roopan P M <v-roopanpm@microsoft.com> * Update test.yml (microsoft#43) * Disabling Text Area functionality (microsoft#47) * fix: Usability and Alignments changes (microsoft#48) * Name update and padding removed * fix home page padding and cards height * remove gap in tasks section * Update docker-build-and-push.yml to debug * Update docker-build-and-push.yml * Update docker-build-and-push.yml to take the correct event name * Update docker-build-and-push.yml * fix: text area background color (microsoft#50) * fix: text area background color * Cursor hover style and Light dark color updated * feat: customize track events (microsoft#54) * feat: customized track event * pylint fix * pylint fix --------- Co-authored-by: Prashant-Microsoft <v-pmalusare@microsoft.com> Co-authored-by: Kiran-Siluveru-Microsoft <v-ksiluveru@microsoft.com> Co-authored-by: Mohan-Microsoft <v-mvenudass@microsoft.com> Co-authored-by: Harmanpreet-Microsoft <v-harmanprka@microsoft.com>
* Update README.md * Update README.md * Update README.md * Update README.md * Update LocalDeployment.md * Update LocalDeployment.md * Update azure_app_service_auth_setup.md * Update LocalDeployment.md * Update README.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information