Skip to content

[52] support "variantKey" parameter in withToggledHookFactory, mirroring withTogglePointFactory - #53

Merged
asos-dominicjomaa merged 19 commits into
ASOS:mainfrom
TomStrepsil:bug/52-support-variantKey-in-with-toggled-hook-factory
Sep 30, 2025
Merged

asos-dominicjomaa merged 19 commits into
ASOS:mainfrom
TomStrepsil:bug/52-support-variantKey-in-with-toggled-hook-factory

Conversation

@TomStrepsil

@TomStrepsil TomStrepsil commented Jul 27, 2025 •

Copy link
Copy Markdown
Contributor

Issue

resolves #52

Details

  • Add variantKey option for withToggledHookFactory, aligning with change made to withTogglePointFactory in version 0.4.0 of the react-pointcuts package.
  • Add a "content management" fixture to the "next" example. that utilises withToggledHook

Scout Rule

  • refactored the webpackNormalModule option of the webpack package to just take a module, rather than an async function - born out of a mistake when importing originally, I assume.
  • Add note re: Ensure nodeRequestScopedFeaturesStoreFactory can create stores shared within a NextJS / multi-realm runtime #50 to the Next example documentation
  • Add a "toggled twice" fixture to the NextJs example
  • removed errant toggle-point.d.ts from tsconfig.json in NextJs example
  • fix JSDoc comment for withTogglePointFactory
  • removed "tags" badge from main README.md, this doesn't make sense for a monorepo that doesn't git-tag releases
  • added a "discussions" badge, akin to the "issues" one

CheckList

  • PR starts with [ISSUE_ID].
  • Has been tested (where required) before merge to main.

TomStrepsil and others added 11 commits December 24, 2024 17:31
* rename to proper module namespace

* update docs links

* update versions

* web toggle point in readme title

* fixup changelog from revised 0.x range

* 2.0.0 -> 0.5.0 in oss version scheme

* fix broken link syntax in CHANGELOG

* consistent quoting

* more version history issues

* fixup module name in jsdoc

* add web
remove sdkInstanceProvider

* remove SDKInstanceProvider

* fixup jsdoc dedupe

* tweak

* clarity re: ssr package

* casing etc
* update workflows

* version

* typo

* update chromium linux snaps

* versions for serve update

* package.json repository field

* update root package.lock

* bugs & directories/doc fields

* fix changelog

---------

Co-authored-by: Tom Pereira <tom.pereira@johnlewis.co.uk>
Co-authored-by: Tom Pereira <tom.pereira@johnlewis.co.uk>
add content management example
fixup webpackNormalModule
fixup package.json for next example
consistent README/specs
@TomStrepsil
TomStrepsil requested a review from a team as a code owner July 27, 2025 13:40
@TomStrepsil TomStrepsil changed the title [52] support variant key in with toggled hook factory [52] support "variantKey" parameter in withToggledFookFactory, mirroring withTogglePointFactory Jul 27, 2025
@TomStrepsil TomStrepsil changed the title [52] support "variantKey" parameter in withToggledFookFactory, mirroring withTogglePointFactory [52] support "variantKey" parameter in withToggledHookFactory, mirroring withTogglePointFactory Sep 26, 2025
@asos-dominicjomaa
asos-dominicjomaa merged commit bf5b973 into ASOS:main Sep 30, 2025
10 checks passed
@TomStrepsil
TomStrepsil deleted the bug/52-support-variantKey-in-with-toggled-hook-factory branch October 21, 2025 09:23
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.

Support "variantKey" option in withToggledHookFactory

2 participants