-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Adding Conversation History, "Vanilla" chat page, and tweaked local dev settings for hot swap #259
Conversation
Not actually keyvault, but adding placeholder ocmponents in JSX for ChatGPT page. Issue/1 add keyvault
added cosmosdb to bicep
…into feat-add-chatgpt-ui-issue13
Feat add chatgpt UI issue13
fixed typo in azure.yaml was breaking azd deploy
Add "Design/Wiki Reference" to ISSUE_TEMPLATE #20
Issue/add codeowners
Change read retrieve to use gp35 #13
Bdl add auth - power MERGE!
…thub.com/bdleavitt/chatgpt-on-azure into 28-as-a-user-i-want-my-chats-to-be-saved
…thub.com/bdleavitt/chatgpt-on-azure into 28-as-a-user-i-want-my-chats-to-be-saved
…-be-saved 28 as a user i want my chats to be saved
…-chats 6 as a user i want to delete my chats
…-chats clear chat on delete
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed. |
Hi @pamelafox , This feature is most asked and I think this is the only feature missing from this repository , Rest everything already covered , Can you add this one as well? Issues #977 #489 , Also this PR will help in fixing may other relatable issue with chat history. @bdleavitt , This pull request needs to be updated since the code has been significantly modified. |
We'll try to get to this in the near future. Currently, our priority is ironing out issues with existing features, particularly user authentication, upload, and GPT-4-vision. But I agree that this would be a good feature to bring this app to feature parity with other RAG solutions. |
@bdleavitt Could you please attach a demo to this PR? We would also like to add this feature in our application |
Thanks @bdleavitt for this PR from a year ago, for inspiring many future PRs! |
Purpose
Deploying module deploys CosmosDB instance for storing conversation history. Uses Azure app service's built-in authentication to separate user conversations from each othe.r
Does this introduce a breaking change?
Pull Request Type
What kind of change does this Pull Request introduce?
How to Test
Initialize repo
azd init
deploy infra
azd up
Enable auth on app service
Open website. Navigate to "ChatGPT" tab.
What to Check
Verify that the following are valid
Other Information