Skip to content

Commit 1c35e36

Browse files
fix: feature flag a11y features in listing pipeline (#5979)
1 parent 29b164e commit 1c35e36

File tree

4 files changed

+32
-10
lines changed

4 files changed

+32
-10
lines changed

api/prisma/seed-helpers/listing-factory.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ export const listingFactory = async (
3939
applicationDueDate?: Date;
4040
closedAt?: Date;
4141
digitalApp?: boolean;
42+
enableListingFeaturesAndUtilities?: boolean;
4243
includeBuildingFeatures?: boolean;
4344
includeEligibilityRules?: boolean;
4445
includeReservedCommunityTypes?: boolean;
@@ -230,10 +231,12 @@ export const listingFactory = async (
230231
: {},
231232
...additionalEligibilityRules(optionalParams?.includeEligibilityRules),
232233
...buildingFeatures(optionalParams?.includeBuildingFeatures),
233-
...featuresAndUtilites(
234-
optionalParams?.optionalFeatures,
235-
optionalParams?.optionalUtilities,
236-
),
234+
...(optionalParams?.enableListingFeaturesAndUtilities
235+
? featuresAndUtilites(
236+
optionalParams?.optionalFeatures,
237+
optionalParams?.optionalUtilities,
238+
)
239+
: {}),
237240
...(optionalParams?.listing?.listingType === ListingTypeEnum.nonRegulated
238241
? listingsRequiredDocuments(optionalParams?.requiredDocumentsList)
239242
: {}),

api/prisma/seed-staging.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -991,6 +991,7 @@ export const stagingSeed = async (
991991
],
992992
userAccounts: [{ id: partnerUser.id }],
993993
optionalFeatures: { carpetInUnit: true },
994+
enableListingFeaturesAndUtilities: true,
994995
},
995996
],
996997
[
@@ -1182,6 +1183,7 @@ export const stagingSeed = async (
11821183
}),
11831184
],
11841185
userAccounts: [{ id: partnerUser.id }],
1186+
enableListingFeaturesAndUtilities: true,
11851187
},
11861188
],
11871189
[
@@ -1210,6 +1212,7 @@ export const stagingSeed = async (
12101212
},
12111213
],
12121214
userAccounts: [{ id: partnerUser.id }],
1215+
enableListingFeaturesAndUtilities: true,
12131216
},
12141217
],
12151218
[
@@ -1250,6 +1253,7 @@ export const stagingSeed = async (
12501253
}),
12511254
],
12521255
userAccounts: [{ id: partnerUser.id }],
1256+
enableListingFeaturesAndUtilities: true,
12531257
},
12541258
],
12551259
[
@@ -1259,6 +1263,7 @@ export const stagingSeed = async (
12591263
listing: littleVillageApartments,
12601264
multiselectQuestions: [workInCityQuestion],
12611265
userAccounts: [{ id: partnerUser.id }],
1266+
enableListingFeaturesAndUtilities: true,
12621267
},
12631268
],
12641269
[
@@ -1404,6 +1409,7 @@ export const stagingSeed = async (
14041409
},
14051410
],
14061411
userAccounts: [{ id: partnerUser.id }],
1412+
enableListingFeaturesAndUtilities: true,
14071413
},
14081414
],
14091415
[
@@ -1439,6 +1445,7 @@ export const stagingSeed = async (
14391445
},
14401446
},
14411447
],
1448+
enableListingFeaturesAndUtilities: true,
14421449
},
14431450
],
14441451
[
@@ -1500,6 +1507,7 @@ export const stagingSeed = async (
15001507
},
15011508
},
15021509
],
1510+
enableListingFeaturesAndUtilities: true,
15031511
},
15041512
],
15051513
];

api/test/integration/listing.e2e-spec.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -685,6 +685,7 @@ describe('Listing Controller Tests', () => {
685685
}),
686686
});
687687
const listing1Input = await listingFactory(jurisdictionB.id, prisma, {
688+
enableListingFeaturesAndUtilities: true,
688689
listing: {
689690
homeType: HomeTypeEnum.apartment,
690691
isVerified: true,
@@ -718,6 +719,7 @@ describe('Listing Controller Tests', () => {
718719
});
719720

720721
const listing2Input = await listingFactory(jurisdictionB.id, prisma, {
722+
enableListingFeaturesAndUtilities: true,
721723
includeReservedCommunityTypes: true,
722724
listing: {
723725
homeType: HomeTypeEnum.duplex,

sites/partners/src/components/listings/PaperListingForm/index.tsx

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -395,14 +395,23 @@ const ListingForm = ({
395395
formData.listingType = undefined
396396
}
397397

398-
if (formData.configurableAccessibilityFeatures) {
399-
setAccessibilityFeatures(
400-
Object.values(formData.configurableAccessibilityFeatures).flat() as string[]
398+
if (
399+
doJurisdictionsHaveFeatureFlagOn(
400+
FeatureFlagEnum.enableAccessibilityFeatures,
401+
jurisdictionId
401402
)
402-
}
403+
) {
404+
if (formData.configurableAccessibilityFeatures) {
405+
setAccessibilityFeatures(
406+
Object.values(formData.configurableAccessibilityFeatures).flat() as string[]
407+
)
408+
}
403409

404-
if (!formData.configurableAccessibilityFeatures) {
405-
formData.configurableAccessibilityFeatures = accessibilityFeatures
410+
if (!formData.configurableAccessibilityFeatures) {
411+
formData.configurableAccessibilityFeatures = accessibilityFeatures
412+
}
413+
} else {
414+
delete formData.configurableAccessibilityFeatures
406415
}
407416

408417
if (successful) {

0 commit comments

Comments
 (0)