Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: event page update #1449

Merged
merged 5 commits into from
Mar 26, 2023
Merged

fix: event page update #1449

merged 5 commits into from
Mar 26, 2023

Conversation

AceTheCreator
Copy link
Member

Sorted the list of events on the event page and changed the title on the community navbar from "Meetings" to "Events."

#917

cc @akshatnema

@netlify
Copy link

netlify bot commented Mar 20, 2023

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 51489c0
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/641841bb57bc75000894d692
😎 Deploy Preview https://deploy-preview-1449--asyncapi-website.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 site settings.

@github-actions
Copy link

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 35
🟠 Accessibility 88
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 30

Lighthouse ran on https://deploy-preview-1449--asyncapi-website.netlify.app/

@derberg
Copy link
Member

derberg commented Mar 20, 2023

83 files changed and targets master - is it correct?

@AceTheCreator AceTheCreator changed the base branch from master to community March 20, 2023 11:49
@AceTheCreator
Copy link
Member Author

83 files changed and targets master - is it correct?

Thanks man 😄

@derberg
Copy link
Member

derberg commented Mar 21, 2023

we have banner in asyncapi/community#640

I though your idea was to use it in https://deploy-preview-1449--asyncapi-website.netlify.app/community/events

@AceTheCreator
Copy link
Member Author

AceTheCreator commented Mar 21, 2023

we have banner in asyncapi/community#640

I though your idea was to use it in https://deploy-preview-1449--asyncapi-website.netlify.app/community/events

Yes, I use this current one you can see in the events as fallbacks when it loads events that don't have a cover.

You can't see it now cuz we haven't merged it with master. The new addition to the build-meeting script helps with that

Copy link
Member

derberg commented Mar 21, 2023

ok for me, @akshatnema if you approve, we can merge

Copy link
Member

@akshatnema akshatnema left a comment

Choose a reason for hiding this comment

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

image

@AceTheCreator I can't see any list of events of the recorded/previous events. Aren't we storing it in some json file?

@AceTheCreator
Copy link
Member Author

image

@AceTheCreator I can't see any list of events of the recorded/previous events. Aren't we storing it in some json file?

You can't see them because they don't exist yet, but the new addition to generate-meeting script is going to ensure that

Copy link
Member

@akshatnema akshatnema left a comment

Choose a reason for hiding this comment

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

All good from my side. Approved ✔️

@akshatnema
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit f1b6515 into asyncapi:community Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants