Skip to content

feat: add image gallery for past conferences#890

Open
kajal-jotwani wants to merge 10 commits into
asyncapi:masterfrom
kajal-jotwani:feat/conference-gallery
Open

feat: add image gallery for past conferences#890
kajal-jotwani wants to merge 10 commits into
asyncapi:masterfrom
kajal-jotwani:feat/conference-gallery

Conversation

@kajal-jotwani

@kajal-jotwani kajal-jotwani commented Dec 22, 2025

Copy link
Copy Markdown

Description

  • Refactored the homepage Gallery section to display a static preview (first 9 images) instead of an auto-rotating carousel
  • Created a dedicated /gallery page to display the full collection of images
  • Updated the "Browse" button to navigate to the new gallery route via useRouter
Screencast.from.2026-01-08.01-02-13.mp4

Related issue(s) - #877

@netlify

netlify Bot commented Dec 22, 2025

Copy link
Copy Markdown

Deploy Preview for peaceful-ramanujan-288045 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 6ff52a3
🔍 Latest deploy log https://app.netlify.com/projects/peaceful-ramanujan-288045/deploys/69c935e599eff20008b78e38
😎 Deploy Preview https://deploy-preview-890--peaceful-ramanujan-288045.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@kajal-jotwani

Copy link
Copy Markdown
Author

@AceTheCreator @thulieblack Could you please take a look and share your thoughts on this? Does this feel good, or should I stick to the earlier discussed approach with few images and a separate gallery page?

@thulieblack

Copy link
Copy Markdown
Member

Hey @kajal-jotwani where does the
image

take us?

I think to have a clean home page its best to stick to a separate page

Copilot AI review requested due to automatic review settings January 6, 2026 10:45

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kajal-jotwani

Copy link
Copy Markdown
Author

@thulieblack It would have pointed to the drive where all the conference Images are present. I'll clean the home page and add a seperate gallery page

@kajal-jotwani

Copy link
Copy Markdown
Author

I have made the changes and updated the description.

@AceTheCreator

Copy link
Copy Markdown
Member

@kajal-jotwani can you share where these images are stored?

@kajal-jotwani

Copy link
Copy Markdown
Author

Hi @AceTheCreator ,
The images are stored in Cloudinary under a dedicated asyncapi folder.
I can share the my environment variables if you’d like to test this on your end.

@AceTheCreator

Copy link
Copy Markdown
Member

Hi @AceTheCreator , The images are stored in Cloudinary under a dedicated asyncapi folder. I can share the my environment variables if you’d like to test this on your end.

I understand... but my question is, how did you get the images you uploaded to Cloudinary?

@kajal-jotwani

Copy link
Copy Markdown
Author

I understand... but my question is, how did you get the images you uploaded to Cloudinary?

I downloaded the images from AsyncAPI Slack and the shared Google Drive, then drag-and-dropped them into Cloudinary via the dashboard.

@AceTheCreator

Copy link
Copy Markdown
Member

I understand... but my question is, how did you get the images you uploaded to Cloudinary?

I downloaded the images from AsyncAPI Slack and the shared Google Drive, then drag-and-dropped them into Cloudinary via the dashboard.

Can you share the Google Drive link 🙏🏽

@kajal-jotwani

Copy link
Copy Markdown
Author

@TenzDelek

Copy link
Copy Markdown
Member

it would be better to store the image in the repo it self like we are doing for other images. remember to use webp format.

@thulieblack

Copy link
Copy Markdown
Member

@TenzDelek, the approach is correct. We want to store the images on the cloud, not on GitHub, for the gallery section. By the way, @kajal-jotwani, are you still interested in working on the PR?

@kajal-jotwani

Copy link
Copy Markdown
Author

@thulieblack Yes, I'm interested in working on it. Thanks!

@thulieblack

Copy link
Copy Markdown
Member

please do resolve the conflicts so we can review the PR

@kajal-jotwani

Copy link
Copy Markdown
Author

please do resolve the conflicts so we can review the PR

I've resolved the merge conflicts now

@thulieblack

Copy link
Copy Markdown
Member

Kajal the gallery section is blank from my end.

@TenzDelek please also help and add this to priority as well

@kajal-jotwani

Copy link
Copy Markdown
Author

Kajal the gallery section is blank from my end.

@thulieblack The gallery images are fetched from Cloudinary, so the API requires the Cloudinary environment variables (CLOUDINARY_CLOUD_NAME, CLOUDINARY_API_KEY, CLOUDINARY_API_SECRET).
The deploy preview doesn’t have these configured, which is why the gallery appears blank. It works locally when the credentials are set.
If you'd like to test it locally, I can share the credentials via Slack DM.

@thulieblack

Copy link
Copy Markdown
Member

@TenzDelek please lead with reviewing this PR please

@TenzDelek

Copy link
Copy Markdown
Member

will look into it during this weekend. do ping me once @kajal-jotwani

@kajal-jotwani

Copy link
Copy Markdown
Author

will look into it during this weekend. do ping me once @kajal-jotwani

Hey @TenzDelek, could you please take a look whenever you get a chance? Thanks!

@Recxsmacx

Copy link
Copy Markdown
Contributor

@kajal-jotwani why browse is not responding it is diverting me to a black page?

@TenzDelek

Copy link
Copy Markdown
Member

can you resolve the conflicts? @kajal-jotwani

@TenzDelek

Copy link
Copy Markdown
Member

any updates on this? If there’s no response within one week of this comment, we’ll treat this PR as abandoned and close it.
cc @kajal-jotwani

@kajal-jotwani

Copy link
Copy Markdown
Author

I'll resolve the conflicts by this week

@netlify

netlify Bot commented Jul 2, 2026

Copy link
Copy Markdown

Deploy Preview for peaceful-ramanujan-288045 failed.

Built without sensitive environment variables

Name Link
🔨 Latest commit bd0290c
🔍 Latest deploy log https://app.netlify.com/projects/peaceful-ramanujan-288045/deploys/6a55c72b4a65f20008e2d1af

@kajal-jotwani

Copy link
Copy Markdown
Author

@TenzDelek I have resolved the conflicts now. The build is currently failing because of the navDrop.tsx type error after syncing with master

@TenzDelek

Copy link
Copy Markdown
Member

@TenzDelek I have resolved the conflicts now. The build is currently failing because of the navDrop.tsx type error after syncing with master

can you resolve that build issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants