Skip to content

Commit

Permalink
erase date extractor as parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
migalko94 committed Oct 17, 2023
1 parent f95266f commit 64ec177
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@ import { dateExtractor, sortByDate } from '../../helpers';

export const sortRolesByDate = (experience: ExperienceVm): ExperienceVm => ({
...experience,
roles: sortByDate(experience.roles, dateExtractor) as Role[],
roles: sortByDate(experience.roles) as Role[],
});

export const mapSortedRolesIntoExperience = (experience: ExperienceVm[]): ExperienceVm[] =>
experience.map(experienceItem => sortRolesByDate(experienceItem));

export const sortExperienceByDate = (experience: ExperienceVm[]): ExperienceVm[] =>
sortByDate(experience, dateExtractor) as ExperienceVm[];
sortByDate(experience) as ExperienceVm[];
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ import { dateExtractor, sortByDate } from '../../helpers';
import { StudiesSectionVm } from './studies-section.vm';

export const sortStudiesByDate = (studies: StudiesSectionVm[]): StudiesSectionVm[] =>
sortByDate(studies, dateExtractor) as StudiesSectionVm[];
sortByDate(studies) as StudiesSectionVm[];
3 changes: 1 addition & 2 deletions packages/manfred-common/src/helpers/date-helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@ export const dateExtractor = (item: StudiesSectionVm | ExperienceVm | Role): num
'roles' in item ? new Date(item.roles[0].startDate).getTime() : new Date(item.startDate).getTime();

export const sortByDate = (
array: StudiesSectionVm[] | ExperienceVm[] | Role[],
dateExtractor: (item: StudiesSectionVm | ExperienceVm | Role) => number
array: StudiesSectionVm[] | ExperienceVm[] | Role[]
): StudiesSectionVm[] | ExperienceVm[] | Role[] =>
array.sort(
(a: ExperienceVm | StudiesSectionVm | Role, b: ExperienceVm | StudiesSectionVm | Role) =>
Expand Down

0 comments on commit 64ec177

Please sign in to comment.