Skip to content

Conversation

easyCZ
Copy link
Member

@easyCZ easyCZ commented Sep 17, 2025

  • Remove configcat-js and configcat-react dependencies
  • Delete src/hooks/use-configcat.ts
  • Remove ConfigCatProvider wrappers from popup and button components
  • Default to Ona branding and behavior (previously behind is_ona_browser_extension_enabled flag)

Description

Related Issue(s)

Fixes #

How to test

Documentation

/hold

easyCZ and others added 3 commits September 17, 2025 13:38
- Remove configcat-js and configcat-react dependencies
- Delete src/hooks/use-configcat.ts
- Remove ConfigCatProvider wrappers from popup and button components
- Default to Ona branding and behavior (previously behind is_ona_browser_extension_enabled flag)

Co-authored-by: Ona <[email protected]>
Removes configcat-js, configcat-react, and configcat-common entries
from the lockfile to match package.json changes.

Co-authored-by: Ona <[email protected]>
- Update @types/node from 20.5.0 to 20.14.5 to match main project
- Update other test dependencies to match main project versions
- Remove test/pnpm-lock.yaml to force regeneration with new dependencies
- Fixes TS2386 error: Overload signatures must all be optional or required

Co-authored-by: Ona <[email protected]>
easyCZ and others added 7 commits September 17, 2025 14:39
- Change headless: "new" to headless: true for Puppeteer v23+ compatibility
- Fix $x method type error by casting page to any and adding proper return type
- Add ElementHandle import for proper typing
- Resolves TS2322 and TS2339 compilation errors

Co-authored-by: Ona <[email protected]>
Co-authored-by: Ona <[email protected]>
Co-authored-by: Ona <[email protected]>
Co-authored-by: Ona <[email protected]>
@easyCZ easyCZ marked this pull request as ready for review September 18, 2025 06:24
@easyCZ easyCZ merged commit 84b2d84 into main Sep 18, 2025
2 checks passed
@easyCZ easyCZ deleted the remove-configcat-feature-flag branch September 18, 2025 06:24
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.

3 participants