Skip to content

Conversation

@Slashgear
Copy link
Contributor

@Slashgear Slashgear commented Jul 31, 2023

Why

We want to add AB test and feature flag frontend tools to many projects at scaleway.
In order to lever what have been done in Console project, I propose to extract the init of AB test in a dedicate package which has the responsability and API of it.

For now, Growthbook is used but this package allow to change it or to enhance/adapt it to our usage and share it within the company.

How

  • create a dedicate package
  • add README to describe the usage of it
  • Ignore Ts errors as GrowthBook JS sdk has some error with the Typescrypt definition export.
  • Facade it to ABTest API (we don't know if growthbook will be their forever)

@changeset-bot
Copy link

changeset-bot bot commented Jul 31, 2023

🦋 Changeset detected

Latest commit: 5fa51b2

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Slashgear Slashgear force-pushed the ab-test-tool branch 2 times, most recently from b2e3fc4 to 6e672d6 Compare July 31, 2023 13:13
@philibea
Copy link
Collaborator

@Slashgear missing some file inside the package:

  • .npmignore
  • .eslintrc.cjs

@Slashgear Slashgear changed the title feat(abtest): add new package to mutualise GrowthBook init feat(growthbook): add new package to mutualise GrowthBook init Jul 31, 2023
@philibea
Copy link
Collaborator

@Slashgear missing some test ?

@philibea philibea requested review from QuiiBz and chambo-e July 31, 2023 14:16
@philibea philibea added the enhancement New feature or request label Jul 31, 2023
@Slashgear
Copy link
Contributor Author

@philibea Just added some test :)

@Slashgear Slashgear requested review from chambo-e and philibea August 2, 2023 14:37
@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Merging #1416 (f41fc10) into main (83d4ede) will increase coverage by 0.03%.
The diff coverage is 100.00%.

❗ Current head f41fc10 differs from pull request most recent head 5fa51b2. Consider uploading reports for the commit 5fa51b2 to get more accurate results

@@            Coverage Diff             @@
##             main    #1416      +/-   ##
==========================================
+ Coverage   98.65%   98.69%   +0.03%     
==========================================
  Files          24       26       +2     
  Lines         669      688      +19     
  Branches       95       97       +2     
==========================================
+ Hits          660      679      +19     
  Misses          1        1              
  Partials        8        8              
Files Changed Coverage Δ
packages/use-growthbook/src/AbTestProvider.tsx 100.00% <100.00%> (ø)
packages/use-growthbook/src/useAbTestAttributes.ts 100.00% <100.00%> (ø)
packages/use-query-params/src/index.ts 100.00% <100.00%> (ø)

@Slashgear Slashgear force-pushed the ab-test-tool branch 2 times, most recently from f41fc10 to ed4906c Compare August 7, 2023 07:42
@Slashgear Slashgear enabled auto-merge (squash) August 7, 2023 08:20
@Slashgear Slashgear merged commit c726608 into scaleway:main Aug 7, 2023
@Slashgear Slashgear deleted the ab-test-tool branch August 7, 2023 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants