Skip to content

Commit

Permalink
Merge pull request #1349 from ASU/ERFI-151
Browse files Browse the repository at this point in the history
fix(app-rfi): update plan title to use market description
  • Loading branch information
mlsamuelson authored Sep 4, 2024
2 parents 20712d9 + 72be88b commit 3af35c3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 10 deletions.
10 changes: 2 additions & 8 deletions packages/app-rfi/src/core/utils/datasource-helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,14 +56,8 @@ export const normalizeDegreeData = data => {
({ categoryDescription }) => categoryDescription
) || [],

// curr.acadPlanDescription string
// title = (Degree not included)
// title (Certificate) = (ex Degree is a Cert)
title: `${curr.acadPlanDescription}${
curr.degreeDescriptionShort && curr.acadPlanType !== KEY.CER
? ` (${curr.degreeDescriptionShort})`
: ``
}`,
// curr.acadPlanMarketingDescription string
title: curr.acadPlanMarketingDescription,
};
acc.push(p);
}
Expand Down
4 changes: 2 additions & 2 deletions packages/app-rfi/src/core/utils/fetchPrograms.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ function getServiceUrl({
if (Interest2) {
serviceUrl =
`${dataSourceDegreeSearch}/acad-plan/${Interest2}` +
`?include=degreeDescriptionShort` +
`?include=acadPlanMarketingDescription` +
`&include=degreeDescriptionLong` +
`&include=campusesOffered` +
`&include=planCategories` +
Expand Down Expand Up @@ -78,7 +78,7 @@ function getServiceUrl({
`${dataSourceDegreeSearch}/acad-plans` +
`?filter=activeInDegreeSearch` +
`${parameter}` +
`&include=degreeDescriptionShort` +
`&include=acadPlanMarketingDescription` +
`&include=degreeDescriptionLong` +
`&include=campusesOffered` +
`&include=planCategories` +
Expand Down

0 comments on commit 3af35c3

Please sign in to comment.