Skip to content

feat: add openim chat env and config#306

Merged
cubxxw merged 2 commits intomainfrom
feat/set-env
Dec 15, 2023
Merged

feat: add openim chat env and config#306
cubxxw merged 2 commits intomainfrom
feat/set-env

Conversation

@cubxxw
Copy link
Contributor

@cubxxw cubxxw commented Dec 15, 2023


🔍 What type of PR is this?

/kind documentation
/kind feature

👀 What this PR does / why we need it:

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

Could you please provide a default environment variable template for openim-chat and also supply the default environment variables? Thank you.

🅰 Which issue(s) this PR fixes:

Fixes #300

Feature: Update Configuration Reading Logic and Set Default Priority for Chat Application #300

📝 Special notes for your reviewer:

🎯 Describe how to verify it

Could you please help facilitate the validation of this PR by CICD?

📑 Additional documentation e.g., RFC, notion, Google docs, usage docs, etc.:

Signed-off-by: Xinwei Xiong (cubxxw) <[email protected]>
@cubxxw cubxxw requested review from a team as code owners December 15, 2023 03:15
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 15, 2023
Signed-off-by: Xinwei Xiong (cubxxw) <[email protected]>
@sweep-ai-deprecated
Copy link

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

@cubxxw cubxxw enabled auto-merge December 15, 2023 03:26
@cubxxw cubxxw added this pull request to the merge queue Dec 15, 2023
Merged via the queue into main with commit 4f4de63 Dec 15, 2023
@cubxxw cubxxw deleted the feat/set-env branch December 15, 2023 03:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Update Configuration Reading Logic and Set Default Priority for Chat Application

3 participants