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

Implement a Dedicated Database Connection Module #192

Open
BrosG opened this issue May 22, 2023 · 0 comments
Open

Implement a Dedicated Database Connection Module #192

BrosG opened this issue May 22, 2023 · 0 comments
Assignees

Comments

@BrosG
Copy link
Contributor

BrosG commented May 22, 2023

Description:
The database connection process is currently directly embedded into the app.js file. This practice can cause difficulties in managing the database connection logic, especially when the application requires multiple different database connections or complex connection configurations.

Recommended Action:
Refactor the database connection setup into a separate module (e.g., dbConnect.js). This module should export a function that establishes a connection to the database and handles all relevant setup and error handling procedures. This function can then be imported and called in the main application file.

Impact:
This change will allow for easier management and modification of the database connection process, making the application more adaptable to future changes in the database setup or architecture.

@ksibisamir ksibisamir linked a pull request May 30, 2023 that will close this issue
@ksibisamir ksibisamir removed a link to a pull request May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants