Skip to content

Commit

Permalink
Merge pull request #130 from lwestfall/dev
Browse files Browse the repository at this point in the history
fix: results label ordering
  • Loading branch information
lwestfall authored Mar 25, 2024
2 parents ddc9b70 + 5d52934 commit a20b51c
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import {
LinearScale,
} from 'chart.js';
import { _DeepPartialArray } from 'chart.js/dist/types/utils';
import { orderBy } from 'lodash-es';
import { BookDto, BookVoteDto, MeetingDto } from '../../../api/models';

@Component({
Expand Down Expand Up @@ -163,7 +162,8 @@ export class MeetingResultsComponent implements OnChanges {
}
}

labels = orderBy(labels, lbl => bookVoteMap.get(lbl)!.points, 'desc');
// Sort labels by points - note, this only changes the sort order of the labels, not the datasets
// labels = orderBy(labels, lbl => bookVoteMap.get(lbl)!.points, 'desc');

labels = labels.map(lbl => {
const voteCounter = bookVoteMap.get(lbl)!;
Expand Down

0 comments on commit a20b51c

Please sign in to comment.