ComputerScienceHouse / ComputerScienceHouse/packet

/admin/freshmen Times Out

Đang mở
#261 1 bình luận 0 reaction 1 người được giao Được @VivianNK nhận Xem trên GitHub
bug good first issue performance
Ngôn ngữ chính
Python
Star
25
Fork
33
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

The `/admin/freshmen` route times out when there are lots of freshmen (~300 in prod, much more in dev). It works running locally with only two freshmen, so the issue seems to be tied to the amount of data that's being processed and rendered.

My guess is that pagination or lazy loading would solve this.

For pagination, I recommend the [flask sqlalchemy paginate function](https://flask-sqlalchemy.palletsprojects.com/en/2.x/api/?highlight=paginate#flask_sqlalchemy.BaseQuery.paginate).

Lazy loading is a little more complex, and might need api changes. TL;DR: rather than loading up all the freshmen, then rendering the page, the lazy load version would defer the data loading till after the page is rendered, so the user could interact with the page while it's still being populated. (I'm probably not using the term quite correctly here, but it's close enough to the idea)

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

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đá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.