bmlt-enabled / bmlt-enabled/bmlt-server

Replace meeting_main.formats with DB relationship

Ouverte
#1,401 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement
Langage dominant
PHP
Étoiles
23
Forks
17
Merge moyen
9 h 11 min
PR mergées (30 j)
7

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.