Commit 670a8a5
authored
docs: document enrollmentsMap 12-month retention limit (#820)
* docs: document enrollmentsMap 12-month retention limit
Because
* The enrollmentsMap targeting attribute includes both active and inactive
enrollment records, but inactive records are permanently deleted after
12 months (365.25 days) from the original enrollment date
* This caused a production issue where a rollout depending on a holdback
via enrollmentsMap started losing clients when the holdback's enrollment
records aged out of the store
* The existing documentation did not mention the retention behavior or the
difference between enrollmentsMap and activeExperiments/activeRollouts
This commit
* Expands the enrollmentsMap entry in the targeting attributes table
* Adds a detailed subsection explaining enrollmentsMap behavior, the
12-month retention limit, and how to handle holdback transitions
* Adds a comparison table showing the differences between the three
enrollment-related targeting attributes
* docs: clarify clone-and-replace pattern for holdback transitions
Because
* Targeting cannot be modified on already-launched experiments/rollouts
This commit
* Replaces "update dependent targeting expressions" with the correct
workflow: clone the dependent experiment with updated targeting,
launch the clone, then end the original
* docs: clarify 12-month limit only applies after experiment ends
Because
* Active enrollments are never cleaned up regardless of age
* The previous wording could be misread as a hard 12-month cap on all
enrollment records
This commit
* Clarifies that enrollmentsMap entries persist indefinitely while the
experiment is live
* Specifies the 12-month cleanup only starts after the experiment ends
* Notes that clients who enrolled earlier are affected first
* Updates the retention column in the comparison table
* docs: remove prescriptive adaptation pattern from enrollmentsMap section
Because
* The docs should describe behavior, not recommend specific workflows
This commit
* Removes the holdback transition example and clone-and-replace guidance
* docs: link to _cleanupOldRecipes() source for 12-month retention
Because
* Review feedback from Beth to link to the actual code
This commit
* Adds a Searchfox link to _cleanupOldRecipes() in ExperimentStore.sys.mjs
* fix: add missing resolved/integrity for webpack in yarn.lock
Because
* The webpack entry in yarn.lock was missing its resolved URL and
integrity hash, causing yarn to install 5.106.0 instead of the
pinned 5.89.0
* webpack 5.106.0 has a breaking change in ProgressPlugin that is
incompatible with webpackbar 5.0.2, failing the build
This commit
* Adds the resolved URL and integrity hash for webpack 5.89.0
* docs: use pinned revision link for _cleanupOldRecipes
Because
* The mozilla-central source link could shift as lines are added/removed
This commit
* Uses a specific revision hash in the Searchfox link so it stays stable1 parent dc21b35 commit 670a8a5
2 files changed
Lines changed: 29 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
169 | 169 | | |
170 | 170 | | |
171 | 171 | | |
172 | | - | |
173 | | - | |
174 | | - | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
175 | 175 | | |
176 | 176 | | |
177 | 177 | | |
| |||
439 | 439 | | |
440 | 440 | | |
441 | 441 | | |
| 442 | + | |
| 443 | + | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
| 449 | + | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
| 453 | + | |
| 454 | + | |
| 455 | + | |
| 456 | + | |
| 457 | + | |
| 458 | + | |
| 459 | + | |
| 460 | + | |
| 461 | + | |
| 462 | + | |
| 463 | + | |
| 464 | + | |
| 465 | + | |
442 | 466 | | |
443 | 467 | | |
444 | 468 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8121 | 8121 | | |
8122 | 8122 | | |
8123 | 8123 | | |
| 8124 | + | |
| 8125 | + | |
8124 | 8126 | | |
8125 | 8127 | | |
8126 | 8128 | | |
| |||
0 commit comments