Skip to content

Merge upstream#1

Merged
jsboige merged 3 commits intojsboige:mainfrom
microsoft:main
Jul 28, 2023
Merged

Merge upstream#1
jsboige merged 3 commits intojsboige:mainfrom
microsoft:main

Conversation

@jsboige
Copy link
Owner

@jsboige jsboige commented Jul 28, 2023

Motivation and Context

Description

Contribution Checklist

Desmond Howard and others added 3 commits July 27, 2023 19:07
### 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.
-->
removes unused packages/commands from our `package.json`. 

### Description

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->
- removes packages not imported into the app
- removes the `depcheck` command as this is a tool that should be run
globally and not included in the project.
we may want to look into having a job run this command periodically to
clean up the packages, but we shouldn't be adding packages that aren't
used anyway.
- removes the `packaage-lock.json` accidentally added in #55

### Contribution Checklist

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

- [x] The code builds clean without any errors or warnings
- [x] 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
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄
### 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.
-->

This PR defaults to a simplified UX experience and allows users more
control by adding feature flag support and a settings dialog to control
feature visibility.

### Description

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->

- Support for dark mode
- Refactored App component to use FluentProvider and add dark/light
theme support.
- Refactored ChatList and ChatListItem components to add action buttons
buttons: ListItemActions component to display edit, download, and share
actions for chat list items.
- Added DeleteChatDialog component to delete a chat (Frontend only,
feature is disabled by default).
- Added TabView component to handle tabbed views in the chat. 
- Added UserSettingsMenu component to trigger settings dialog. 
- Introduced SettingsDialog component to control feature visibility,
with support for feature descriptions and disabled / inactive features.

![image](https://github.com/microsoft/chat-copilot/assets/125500434/f49755d5-d6f5-4ca9-a54a-043f3e392a70)

![image](https://github.com/microsoft/chat-copilot/assets/125500434/c2faa74b-03a8-424e-832d-762e96e307d5)

![image](https://github.com/microsoft/chat-copilot/assets/125500434/b58b8e7b-db4e-472c-b81e-28c84f47bb6c)

Dark Mode

![image](https://github.com/microsoft/chat-copilot/assets/125500434/0b85a5b8-216b-495f-ab91-7dcc56975520)

### Contribution Checklist

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

- [x] The code builds clean without any errors or warnings
- [x] 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
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: GitHub Actions <actions@github.com>
### 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 persona tab to show/enable the following:
1. Meta prompt.
2. Meta prompt editing,
3. Memory (long term & short term) content.
4. Memory bias slider.

### Description
1. Webapi support for editing the meta prompt.
2. Webapi support for retrieving memory content (ChatMemoryController).
3. Webapi support for setting memory bias.
4. Webapp support for showing and enabling the features.
5. Update the initial bot greeting message.

![image](https://github.com/microsoft/chat-copilot/assets/12570346/8ac7f817-bcab-4b71-98c7-03f3afc3b8f9)

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->

### 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 😄
@jsboige jsboige merged commit 0cb9926 into jsboige:main Jul 28, 2023
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.

3 participants