Skip to content

Create Backend Config Manager#614

Merged
Brain256 merged 20 commits intomainfrom
briany/create-config-manager
Nov 7, 2025
Merged

Create Backend Config Manager#614
Brain256 merged 20 commits intomainfrom
briany/create-config-manager

Conversation

@Brain256
Copy link
Contributor

@Brain256 Brain256 commented Nov 7, 2025

Purpose

Closes #335
Create a BackendConfiguration class to store backend config info (logger excluded params and CORS config settings)

New Changes

  • added .*.env to the .gitignore
  • created Pydantic classes to represent CORS and logger configurations
  • created a BackendConfiguration class to store CORS and logger config settings
  • changed middleware and setup files to take info from an instance of the BackendConfiguration class

Testing

Explain tests that you ran to verify code functionality.

  • I have unit-tested this PR. Otherwise, explain why it cannot be unit-tested.
  • I have included screenshots of the tests performed below.
image

Outstanding Changes

If there are non-critical changes (i.e. additional features) that can be made to this feature in the future, indicate them here.

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
Adityya-K
🥇
56
▀▀▀▀▀
1d 4h 3m
180
▀▀▀▀
camspec
🥈
27
▀▀▀
1d 16h 6m
168
▀▀▀▀
kepler452b123
🥉
15
6d 7h 42m
38
Yarik-Popov
4
1d 1h 41m
40
proprogrammer504
2
7d 1h 31m
▀▀
7
Navtajh04
1
1d 3h 20m
0
sunray4
1
12d 21m
▀▀▀
9
c4bae
1
13d 1h 35m
▀▀▀
2

⚡️ Pull request stats

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very well done! Some small comments and this looks good to me

camspec
camspec previously approved these changes Nov 7, 2025
Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Brain256 Brain256 merged commit 98dafba into main Nov 7, 2025
49 checks passed
@Brain256 Brain256 deleted the briany/create-config-manager branch November 7, 2025 03:24
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.

Create configuration manager

2 participants