refactor(get-ahblines): Get ahblines from sqlite instead from json files #416
Conversation
hf-krechan
changed the title
There was a problem hiding this comment.
vllt schreibst du , nur als kommentar für uns, noch rein, wo diese spalten in dieser form herkommen. das ist janur der prototypische stand mit dem ich es gestern einmal reingehackt habe. ich würde ich der vollständigkeit und reproduzierbarkeit einfach zumindest referenzieren: https://github.com/Hochfrequenz/ahb-mig-backend/releases/tag/v0.0.1 ganz unabhängig davon ob es in diesem repo bleibt oder nicht oder wo es in zukunft herkommt: so hat man die chance die DB reproduzierbar nachzubauen, zumindest als jemand mit zugriff auf private HF repos.
There was a problem hiding this comment.
done in e835315
| if (type === FileType.JSON) { | ||
| return this.getFromDatabase(pruefi, formatVersion); | ||
| } else { | ||
| return this.getFromBlobStorage(pruefi, formatVersion, type); | ||
| } |
There was a problem hiding this comment.
das ist aber langfristig schwer zu maintainen, oder? vermute mal, das wird in einem folge-pr geändert.
There was a problem hiding this comment.
Wollte das feature mit den excel-files noch weiter anbieten.
Wo siehst du Bedenken?
src/server/repository/ahb.ts
Outdated
| meta: { | ||
| description: metaInfo.description || '', | ||
| direction: metaInfo.direction || '', | ||
| maus_version: '0.3.1', // TODO: Add this to the database schema |
There was a problem hiding this comment.
braucht das der lcient oder irgendwer? Ich würde das einfach auslassen
There was a problem hiding this comment.
nö brauchen wir nicht
There was a problem hiding this comment.
done in 4698ec8
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
review #415 first.