Sorting tables

[JochemVH1]

#300: Added sorting for tables, still missing sorting on invoice and invoice months

[Laoujin]

Ik zou zelfs iets verder gegaan zijn en hier gezet hebben sort: true

En dan logic ala:

if (sort === true) {
   // sort on element[key]
  // if (typeof element[key] === 'string') --> .localeCompare sort
}

Maar laat dit maar zo, expliciet is ook OK.

Het voordeel van de sort: true zou vb zijn dat je op een bepaald moment kan kiezen om de .localeCompare door iets complexer te vervangen (vb: met een latinize erbij)

[Laoujin]

In TypeScript kun je dingen strongly typed maken zonder dat je constants nodig hebt enzo:

type sort = {
  columnName: string;
  direction: 'asc' | 'desc';
}

Of uitsplitsen:

type SortDirection = 'asc' | 'desc';
type sort = {
  columnName: string;
  direction: SortDirection ;
}

[JochemVH1]

ik had die constants gedaan omdat ik af en toe zelf 'asc' had gebruikt in de code en dan had ik geen magic strings ^^

[JochemVH1]

en het type zelf laten genereren door chatGTP

[Laoujin]

type Direction = 'asc' | 'desc';

const dir: Direction = 'asc';
const dir2: Direction = 'asc2'; // compile error

Het zijn geen magic strings, je kan aan een Direction enkel 'asc' of 'desc' toekennen, niets anders
Je intellisense gaat dit ook direct aangeven dat dat je 2 opties zijn.

Dit zijn wel magic strings:

const magic1: string = "asc";
const magic2: string = "asc2";
const magic3 = "asc";