fix: speakers images #400
fix: speakers images #400
Conversation
Aryan4884
requested review from
AceTheCreator,
thulieblack and
Mayaleeeee
as code owners
✅ Deploy Preview for peaceful-ramanujan-288045 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Please ignore multiple commit as I have forgotten to make a different branch for this separate issue, thatswhy my previous commit which was for different issue are also getting shown. |
|
Hey @Aryan4884 please follow the pr naming conventions as mentioned by the |
|
Is it ok now? |
|
@AceTheCreator changed the format to .webp |
|
@Aryan4884 there is some error, please resolve it. In my opinion its arising from the |
…ce-website into second_commit
|
@ashmit-coder By mistake i messed up the file directories and submodules. |
|
@Aryan4884 please resolve the merge conflicts. |
|
@ashmit-coder done |
|
@Aryan4884, kindly resolve the conflict in this PR. we should be good to go once that's fixed 👍🏾 |
|
@AceTheCreator done |
#396
This PR resolves that issue by downloading and storing the speaker images locally. This change ensures that the speaker images are always available, thereby improving the overall reliability and consistency of the website.
Changes Made:
Speaker Images Storage:
Downloaded all speaker images from external URLs and stored them in the /public/img/speakers/ directory.
Updated the speaker image paths to reference the local files instead of external URLs.
Speaker Component:
Updated the image src in the Speaker component to use the new local paths.
Benefits:
Reliability: The images are no longer dependent on external URLs, preventing broken image issues caused by changes in external sources.
Performance: Local images reduce latency and improve load time, since they are served directly from our server.
Maintenance: Easier management of speaker images, as they are all stored and versioned within the project.