bmlt-enabled / bmlt-enabled/bmlt-server

Replace meeting_main.formats with DB relationship

Đang mở
#1,401 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
enhancement
Ngôn ngữ chính
PHP
Star
23
Fork
17
Merge trung bình
9 giờ 11 phút
Pull request đã merge (30 ngày)
7

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.