bmlt-enabled / bmlt-enabled/bmlt-server

Replace meeting_main.formats with DB relationship

Open
#1,401 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
PHP
Stars
23
Forks
17
Avg merge
9h 11m
Merged PRs (30d)
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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.