fix: Keep the Admin.py as uppercase naming to allow the streamlit pick it as is.

Azure-Samples / chat-with-your-data-solution-accelerator

A Solution Accelerator for the RAG pattern running in Azure, using Azure AI Search for retrieval and Azure OpenAI large language models to power ChatGPT-style and Q&A experiences. This includes most common requirements and best practices.

https://azure.microsoft.com/products/search

MIT License

651 stars 324 forks source link

fix: Keep the Admin.py as uppercase naming to allow the streamlit pick it as is. #912

Closed gaurarpit closed 1 month ago

gaurarpit commented 1 month ago

Purpose

Fix the application right now by having Admin.py as is.
Does this introduce a breaking change?
[ ] Yes
[x] No

github-actions[bot] commented 1 month ago

Coverage Report •

File	Stmts	Miss	Cover	Missing
code/backend
Admin.py	21	21	0%	1–5, 7, 9–10, 12, 15–16, 18–19, 22, 29, 36, 39–41, 43, 45
TOTAL	2383	673	71%

Tests	Skipped	Failures	Errors	Time
194	0 :zzz:	0 :x:	0 :fire:	12.514s :stopwatch:

gaurarpit commented 1 month ago

Could you change the PR title to be more descriptive, it will end up on the changelog

Updated. thanks. good shout.

Azure-Samples / chat-with-your-data-solution-accelerator

fix: Keep the Admin.py as uppercase naming to allow the streamlit pick it as is. #912

Purpose

Does this introduce a breaking change?