docs: add explicit provider registration of new services in prisma recipe #2912
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
micalevisk
reviewed
Nov 30, 2023
content/recipes/prisma.md
Outdated
| @@ -391,6 +391,21 @@ export class PostService { | |||
|
|
|||
| Your `UserService` and `PostService` currently wrap the CRUD queries that are available in Prisma Client. In a real world application, the service would also be the place to add business logic to your application. For example, you could have a method called `updatePassword` inside the `UserService` that would be responsible for updating the password of a user. | |||
|
|
|||
| We need to register the new services with Nest so that it can perform the injection. We do this by editing our module file (`app.module.ts`) and adding the new services to the `providers` array of the `@Module()` decorator. | |||
|
|
|||
| ```typescript | |||
There was a problem hiding this comment.
I'm not sure if we need to show any code for this here as registering providers is pretty standard in nestjs apps
There was a problem hiding this comment.
@micalevisk I followed along the recipe, tried to run the app and ran into the error. What do you think about just mentioning it and leaving out the code?
There was a problem hiding this comment.
yeah we can have that mention to recall the dev, but the code itself is redundant here
There was a problem hiding this comment.
I adapted the change accordingly. Does it work like this for you?
micalevisk
approved these changes
Nov 30, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?