bmlt-enabled / bmlt-enabled/bmlt-server

Replace meeting_main.formats with DB relationship

Aperta
#1,401 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
PHP
Stelle
23
Fork
17
Merge medio
9h 11m
PR unite (30g)
7

Descrizione

There is an m:n relationship between meetings and formats, but this is not represented in the DB. Instead of that, meeting_main.formats is a comma separated list.

This not only makes the search code unnecessarily complicated, but results in a big performance hit for larger DBs (and, I assume, for aggregator).

Using Laravel's wonderful migration mechanism and ORM, this should be relatively straightforward to clean up. And, it's a pure refactoring, so we have the tests.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.