/employee/employee-view/ is slow to load as it verifies the existence of the profile pictures

[federoitman]

Bug Report

Description

/employee/employee-view/ takes a load time to load, specially if you use s3 for storage of profile pictures.
A lot of time is spent of the get_avatars method
https://github.com/horilla-opensource/horilla/blob/master/employee/models.py#L207-L218

Steps to Reproduce

1. Load 100 employees with their respective profile pictures
2. Go to /employee/employee-view/
3. Measure the time it takes to load

Expected Behavior

/employee/employee-view/ loads fast

Actual Behavior

/employee/employee-view/ takes a long time to load

Screenshots

Environment

• Latest dev on docker-compose

Possible Solution

Avoid validating the existence of the files and resolve a 404 for the profile picture in the browser

[horilla-opensource]

Hi @federoitman ,

Thank you for reporting the issue. We appreciate your detailed description of the problem.

After reviewing the situation, it seems the load time is impacted by the size of profile pictures, especially when using S3 for storage. We plan to address this by updating the file upload operation. Moving forward, we will compress the file before saving it, which should help reduce load times and improve overall performance.

In the meantime, large file sizes may continue to cause delays in page loading, but this improvement should help resolve the issue.

Thanks again for bringing this to our attention. We’ll keep you updated on the progress of this fix!

Best Regards,
Team Horilla

[federoitman]

I believe it's not only related to the file size. Checking that the files exists in s3 imply hundreds of api calls from the backend to aws before loading the page.