Skip to content

Commit 1b9fb91

Browse files
yushinjEjPlatzer
andauthored
Apply suggestions from code review
Co-authored-by: Evan Platzer <evan.platzer@gordon.edu>
1 parent 06b2b18 commit 1b9fb91

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

src/services/transcript.ts

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -70,15 +70,13 @@ const categorizeItems = async (memberships: MembershipHistory[], jobs: StudentEm
7070

7171
groupedMembershipHistory.activities = memberships;
7272

73-
/**
74-
* Sorting job titles by alphabetical order.
75-
* Sorting job titles by end date if there are the same job titles.
76-
*/
77-
groupedMembershipHistory.experiences.sort((a, b) =>
78-
getJobTitle(a).localeCompare(getJobTitle(b)) !== 0
79-
? getJobTitle(a).localeCompare(getJobTitle(b))
80-
: getExperienceEndDate(b) - getExperienceEndDate(a),
81-
);
73+
// Sort experiences by experience name then by end date
74+
groupedMembershipHistory.experiences.sort((a, b) => {
75+
const nameComparison = getName(a).localeCompare(getName(b));
76+
return nameComparison !== 0
77+
? nameComparison
78+
: getExperienceEndDate(b) - getExperienceEndDate(a);
79+
});
8280

8381
return groupedMembershipHistory;
8482
};
@@ -148,5 +146,5 @@ const transcriptService = {
148146

149147
export default transcriptService;
150148

151-
const getJobTitle = (experience: MembershipHistory | StudentEmployment) =>
149+
const getName = (experience: MembershipHistory | StudentEmployment) =>
152150
'Sessions' in experience ? experience.ActivityCode : experience.Job_Title;

0 commit comments

Comments
 (0)