Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add plans tab #59

Merged
merged 4 commits into from
Jul 28, 2023
Merged

Add plans tab #59

merged 4 commits into from
Jul 28, 2023

Conversation

TaoChenOSU
Copy link
Collaborator

Motivation and Context

Add a plans tab to show created plans..

Description

  1. Add a plans tab.
  2. Add a plan json viewer to allow users to see the plan written in Json.
    image
    image

Contribution Checklist

@TaoChenOSU TaoChenOSU added webapp Pull requests that update Typescript code webapi Pull requests that update .net code PR: ready for review labels Jul 27, 2023
@TaoChenOSU TaoChenOSU self-assigned this Jul 27, 2023
@crickman crickman self-requested a review July 27, 2023 22:49
crickman
crickman previously approved these changes Jul 27, 2023
View reviewed changes
alliscode
alliscode previously approved these changes Jul 27, 2023
View reviewed changes
@TaoChenOSU TaoChenOSU dismissed stale reviews from alliscode and crickman via 4939bda July 28, 2023 00:25
dehoward
dehoward previously approved these changes Jul 28, 2023
View reviewed changes
@TaoChenOSU TaoChenOSU requested a review from dehoward July 28, 2023 15:43
dehoward
dehoward previously approved these changes Jul 28, 2023
View reviewed changes
@dehoward dehoward added PR: ready to merge PR has been approved by all reviewers, and is ready to merge. and removed PR: ready for review labels Jul 28, 2023
@hathind-ms hathind-ms added this pull request to the merge queue Jul 28, 2023
Merged via the queue into microsoft:main with commit d76ab3e Jul 28, 2023
8 checks passed
teresaqhoang pushed a commit to teresaqhoang/chat-copilot that referenced this pull request Jul 31, 2023
@TaoChenOSU @teresaqhoang
Add plans tab (microsoft#59)
322be7c 
### Motivation and Context

<!-- Thank you for your contribution to the copilot-chat repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->
Add a plans tab to show created plans..

### Description

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->
1. Add a plans tab.
2. Add a plan json viewer to allow users to see the plan written in
Json.

![image](https://github.com/microsoft/chat-copilot/assets/12570346/567804be-208f-4a1a-8759-961360808f1c)

![image](https://github.com/microsoft/chat-copilot/assets/12570346/83e764d5-f471-4e1b-ab91-a0afe889b06c)

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄
teamleader-dev pushed a commit to vlink-group/chat-copilot that referenced this pull request Oct 7, 2024
@TaoChenOSU
Add plans tab (microsoft#59)
519cfe2 
### Motivation and Context

<!-- Thank you for your contribution to the copilot-chat repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->
Add a plans tab to show created plans..

### Description

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->
1. Add a plans tab.
2. Add a plan json viewer to allow users to see the plan written in
Json.

![image](https://github.com/microsoft/chat-copilot/assets/12570346/567804be-208f-4a1a-8759-961360808f1c)

![image](https://github.com/microsoft/chat-copilot/assets/12570346/83e764d5-f471-4e1b-ab91-a0afe889b06c)

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dehoward dehoward dehoward approved these changes

@crickman crickman crickman left review comments

@alliscode alliscode alliscode left review comments

@teresaqhoang teresaqhoang Awaiting requested review from teresaqhoang

Assignees

@TaoChenOSU TaoChenOSU

Labels
PR: ready to merge PR has been approved by all reviewers, and is ready to merge. webapi Pull requests that update .net code webapp Pull requests that update Typescript code
Projects
No open projects
Apps & Services Semantic Kernel
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@TaoChenOSU @alliscode @dehoward @crickman @hathind-ms