Skip to content

Conversation

DavidGoestemeier
Copy link
Contributor

Add a comma to the reducer function parameter of the createFeature example in the store docs.

Closes #4967

Docs

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Fix faulty documenation of createFeature.

Closes #4967

What is the new behavior?

Just added a comma to fix the snippet.

Does this PR introduce a breaking change?

[ ] Yes
[X] No

Copy link

netlify bot commented Oct 6, 2025

Deploy Preview for ngrx-io-v18 canceled.

Name Link
🔨 Latest commit 3c33f42
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-io-v18/deploys/68e38e70b25c530008db6a93

Copy link

netlify bot commented Oct 6, 2025

Deploy Preview for ngrx-io ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 12f40a8
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-io/deploys/68e3c6c187c4cb0008138c57
😎 Deploy Preview https://deploy-preview-4972--ngrx-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

* on(ProductsApiActions.loadSuccess, (state, { products }) => ({
* ...state,
* products,
* }),
Copy link
Member

Choose a reason for hiding this comment

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

Thanks @DavidGoestemeier! Can you also add a missing closing bracket:

Suggested change
* }),
* })),

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great catch. Will do. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Add a missing comma and bracket to the createFeature example in the store docs.

Closes ngrx#4967
@DavidGoestemeier DavidGoestemeier force-pushed the fix(#4967)-feature-creator-docs branch from 5ee7fbb to b97da08 Compare October 6, 2025 09:32
Add a missing comma and bracket to the createFeature example in the store docs.

Closes ngrx#4967
@DavidGoestemeier
Copy link
Contributor Author

DavidGoestemeier commented Oct 6, 2025

I guess someone needs to clear yarn cache, because for some odd reason the install failed
I just spent a few minutes trying to find a rerun all jobs option, but didn't find one, probably because of missing permissions.
I am unsure on how to deal with this now.

I could ofc make a empty commit or 2 amends with 1 force push... ah idk I need to work :(

@markostanimirovic markostanimirovic changed the base branch from 18.1.x to main October 6, 2025 13:39
Copy link

netlify bot commented Oct 6, 2025

Deploy Preview for ngrx-site-v19 ready!

Name Link
🔨 Latest commit 12f40a8
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-site-v19/deploys/68e3c6c125164f00080cb264
😎 Deploy Preview https://deploy-preview-4972--ngrx-site-v19.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@markostanimirovic markostanimirovic changed the title docs(store): add missing comma to createFeature reducer docs example docs(store): add missing comma to createFeature example Oct 6, 2025
@markostanimirovic markostanimirovic merged commit 7f35200 into ngrx:main Oct 6, 2025
14 checks passed
@markostanimirovic
Copy link
Member

Thanks @DavidGoestemeier!

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.

NgRx V18 - Documenation for CreateFeature has a small but important Error

3 participants