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.

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

Align AzureBlobs:Container to chatmemory in memorypipeline's settings #1077

Merged
merged 18 commits into from
Jul 24, 2024
Merged

Align AzureBlobs:Container to chatmemory in memorypipeline's settings #1077

merged 18 commits into from
Jul 24, 2024

Conversation

glahaye
Copy link
Contributor

@glahaye glahaye commented Jul 24, 2024

Motivation and Context

The value for AzureBlobs:Container differs between webapi and memorypipeline in the appsettings.json file, which prevents the two from working together.

Description

Align AzureBlobs:Container to chatmemory in memorypipeline's settings

Contribution Checklist

@glahaye glahaye merged commit ddc5d46 into microsoft:main Jul 24, 2024
7 checks passed
@glahaye glahaye deleted the mem-settings branch July 25, 2024 00:54
teamleader-dev pushed a commit to vlink-group/chat-copilot that referenced this pull request Oct 7, 2024
…microsoft#1077)

### Motivation and Context
The value for AzureBlobs:Container differs between webapi and
memorypipeline in the appsettings.json file, which prevents the two from
working together.

### Description
Align AzureBlobs:Container to chatmemory in memorypipeline's settings

### Contribution Checklist
- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/chat-copilot/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 😄
kb0039 pushed a commit to aaronba/chat-copilot that referenced this pull request Jan 8, 2025
…microsoft#1077)

### Motivation and Context
The value for AzureBlobs:Container differs between webapi and
memorypipeline in the appsettings.json file, which prevents the two from
working together.

### Description
Align AzureBlobs:Container to chatmemory in memorypipeline's settings

### Contribution Checklist
- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/chat-copilot/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
Labels
None yet
Projects
None yet
1 participant