Bump FAPI to include Test models - #28992
Open
abeddow91 wants to merge 4 commits into
Open
Conversation
Contributor
11 tasks
abeddow91
force-pushed
the
ab/bump-fapi-to-include-ab-test
branch
3 times, most recently
from
July 28, 2026 19:06
e237e8d to
66a79f6
Compare
simonbyford
force-pushed
the
ab/bump-fapi-to-include-ab-test
branch
from
July 30, 2026 07:13
b64846d to
2e30a79
Compare
abeddow91
force-pushed
the
ab/bump-fapi-to-include-ab-test
branch
from
August 3, 2026 14:41
2e30a79 to
08e0cfd
Compare
This version includes the editorial ab test models
abeddow91
force-pushed
the
ab/bump-fapi-to-include-ab-test
branch
from
August 10, 2026 16:14
0e09b97 to
724b950
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this change?
Allows editorial tests set in facia tool to be pressed to S3 on the pressed page model, which consequently gets sent to DCR during render. This PR also includes Test data is also included on the lite.json enpoint which is polled by DAE. For this endpoint, the tests model has been reduced to only include the following fields,
testUuid,variantMeta,startDate,endDate,hasManuallyEndedOnThisTrailThis PR bumps FAPI to "37.0.0", the version which includes the editorial ab test models.
As the Tests field on trails and supportingItems is an optional field, this bump is backwards compatible and should not affect existing front pressing or rendering
Screenshots
Tests on pressed page
Minimal tests on lite.json