Skip to content

fix: make WithConnectedCount usage incrementable#309

Open
narutamaaurum wants to merge 1 commit into
piotrwitek:masterfrom
narutamaaurum:fix-211-connected-count-usage
Open

fix: make WithConnectedCount usage incrementable#309
narutamaaurum wants to merge 1 commit into
piotrwitek:masterfrom
narutamaaurum:fix-211-connected-count-usage

Conversation

@narutamaaurum

@narutamaaurum narutamaaurum commented May 23, 2026

Copy link
Copy Markdown

Summary

  • remove the hard-coded overrideCount={5} from the WithConnectedCountUsage example
  • correct the example label to match the connected HOC example
  • regenerate the README output so the docs show the working usage snippet

Validation

  • npm run ci-check
  • cd playground && CI=true npm run tsc
  • cd playground && npm run lint

Closes #211

- stop pinning the example to a constant overrideCount
- correct the example label
- regenerate README output

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the usage examples for the FCCounterWithConnectedCount component in both the README and the playground source code by removing the overrideCount prop and correcting the label prop. I have no feedback to provide as there are no review comments.

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.

why the WithConnectedCountUsage component cannot increment the counter?

1 participant