Skip to content

Change the DB interface to allow multiple implementations of the modules. Personality Interface.#1250

Closed
caneppelevitor wants to merge 8 commits intostagefrom
nestjs-multidb-start-personality
Closed

Change the DB interface to allow multiple implementations of the modules. Personality Interface.#1250
caneppelevitor wants to merge 8 commits intostagefrom
nestjs-multidb-start-personality

Conversation

@caneppelevitor
Copy link
Collaborator

@caneppelevitor caneppelevitor commented Jun 13, 2024

No description provided.

- Adapted the app module to accept multi-DB logic
- Created a new config variable and config file to set up DB type
- Adapted personality files to support multi-DB
- Created a personality provider to manage service selection
- Created interfaces (personality and service) using Zod
- Created a mongo folder to hold files for MongoDB logic
- Fixed a spelling error in "skipped"
@thesocialdev thesocialdev changed the title MultiDB Start Personality Mongo Change the DB interface to allow multiple implementations of the modules Jun 13, 2024
- Removed unused env variable
- Removed ConfigModule and ThrottlerModule from if
- Exported in the declaration of interfaces
- Properly organized personality module
- Removed forgotten test lines on personality service
- Removed duplication on stats service
- Removed TODOS
@sonarqubecloud
Copy link

@caneppelevitor caneppelevitor changed the title Change the DB interface to allow multiple implementations of the modules Change the DB interface to allow multiple implementations of the modules. Personality Interface. Jun 13, 2024
@thesocialdev thesocialdev force-pushed the stage branch 7 times, most recently from 0ae3b52 to 20bbdca Compare August 25, 2024 18:22
@thesocialdev thesocialdev force-pushed the stage branch 6 times, most recently from 05f8912 to 864c35e Compare November 7, 2024 18:27
@sonarqubecloud
Copy link

@thesocialdev
Copy link
Collaborator

@caneppelevitor one last rebase/merge and I'll merge it!

@caneppelevitor
Copy link
Collaborator Author

@caneppelevitor one last rebase/merge and I'll merge it!

This branch was cherry-picked into another branch and merged. I think we can delete this PR.
#1802

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants