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

feat: hacktoberfest Blog Post #1131

Merged
merged 24 commits into from
Dec 13, 2022
Merged

feat: hacktoberfest Blog Post #1131

merged 24 commits into from
Dec 13, 2022

Conversation

AceTheCreator
Copy link
Member

I wrote a post summarising our participants at this year's Hacktoberfest

cc @derberg @alequetzalli

@netlify
Copy link

netlify bot commented Nov 28, 2022

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 072ebfb
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/6397ced65f96ca0009fe02b7
😎 Deploy Preview https://deploy-preview-1131--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.

@AceTheCreator AceTheCreator changed the title feat: Hacktoberfest Blog Post feat: hacktoberfest Blog Post Nov 28, 2022
@github-actions
Copy link

github-actions bot commented Nov 28, 2022

⚡️ Lighthouse report for the changes in this PR:

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

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

Comment on lines 46 to 58
<table>
<tr>
<td align="center">
<a href="https://github.com/Amishakumari544">
<img src="https://avatars.githubusercontent.com/u/49230384?v=4" width="100px;" alt=""/>
<br />
<sub>
<b>
Amishakumari544 - 1 PR
</b>
</sub>
</a>
</td>
Copy link
Member

Choose a reason for hiding this comment

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

I don't wanna make review for md content (please wait for @alequetzalli), but maybe will be better to create React component for that, hmm, wdyt? 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe it should come in a separate PR?

Copy link
Member

Choose a reason for hiding this comment

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

separate PR is the biggest lie of a person on open source 😄 I'll fix it right away but in separate PR and PR will never be created XD That's only mu suggestion, you can create or not if you don't want. I don't have a problem with it.

Copy link
Member Author

Choose a reason for hiding this comment

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

I suggested this because I also have a post on the conference where I'll be giving a special shoutout to the volunteers and I'd eventually be listing them like the one above... So the PR will eventually be created 😄

Copy link
Member

@magicmatatjahu magicmatatjahu Nov 28, 2022

Choose a reason for hiding this comment

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

OK, so PR will be (eventually) created, nice 😄

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.

Sorry, I'm not a md reviewer 😅 , but I found some bugs inside the blog. Kindly go through it.

@@ -0,0 +1,207 @@
---
title: "Is Hacktoberfest Good For Maintainers?"
Copy link
Member

Choose a reason for hiding this comment

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

Is this the correct title for this PR? 🤔 Because in the blog post, you are summarising Hacktoberfest 2022 in AsyncAPI.

Copy link
Member

Choose a reason for hiding this comment

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

yup, i think it is title of old existing article

Copy link
Member Author

Choose a reason for hiding this comment

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

I copied from the old one with updating in mind and I forgot 😄

pages/blog/hacktoberfest-2022.md Show resolved Hide resolved
@@ -0,0 +1,207 @@
---
title: "Is Hacktoberfest Good For Maintainers?"
date: 2020-11-05T06:00:00+01:00
Copy link
Member

Choose a reason for hiding this comment

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

The date written is incorrect. Kindly change it.


<table>
<tr>
<td align="center">
Copy link
Member

Choose a reason for hiding this comment

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

image

The table created is scrollable in terms of responsiveness. I would you to use flexbox or make a separate component to show this content.

Copy link
Member

Choose a reason for hiding this comment

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

yo, why not just make a normal markdown table? O_O it's a good practice (unlike this current mess haha) and you don't need to build a new component 😄
Screen Shot 2022-12-02 at 3 42 13 PM

👉🏽👉🏽 here's a simple tool to generate your markdown tables: https://www.tablesgenerator.com/markdown_tables

Copy link
Member Author

Choose a reason for hiding this comment

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

I feel using a component gives us more flexibility in terms of looks. Btw, I already created a PR

<tr>
<td align="center">
<a href="https://github.com/Amishakumari544">
<img src="https://avatars.githubusercontent.com/u/49230384?v=4" width="100px;" alt=""/>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<img src="https://avatars.githubusercontent.com/u/49230384?v=4" width="100px;" alt=""/>
<img src="https://avatars.githubusercontent.com/Amishakumari544" width="100px;" alt=""/>

I think directly using Github username in the URL to show the profile photos is a better option.

@derberg
Copy link
Member

derberg commented Dec 1, 2022

Thanks so much for writing this up 🙏🏼

So feedback:

Maybe mention all the contributors in this article, so they can verify on their own. I mean specify how many hacktoberfest contributions they did

@AceTheCreator
Copy link
Member Author

@Amishakumari544 @toukirkhan @manavdesai27 @Ankitchaudharyy @nawed2611 @dhfherna @amit-ksh @mastdev @codingtenshi @Mayaleeeee Please kindly confirm if the numbers of your PRs during hacktoberfest is right

@toukirkhan
Copy link
Contributor

toukirkhan commented Dec 2, 2022

Yes mine is 2.

@Ankitchaudharyy
Copy link
Contributor

I had 3 PR merged in hacktoberfest.

@Amishakumari544
Copy link
Contributor

@Amishakumari544 @toukirkhan @manavdesai27 @Ankitchaudharyy @nawed2611 @dhfherna @amit-ksh @mastdev @codingtenshi @Mayaleeeee Please kindly confirm if the numbers of your PRs during hacktoberfest is right

yes 🙌

@amit-ksh
Copy link

amit-ksh commented Dec 2, 2022

Yes, 1 PR merged during hacktoberfest

@mastdev
Copy link

mastdev commented Dec 2, 2022

@Amishakumari544 @toukirkhan @manavdesai27 @Ankitchaudharyy @nawed2611 @dhfherna @amit-ksh @mastdev @codingtenshi @Mayaleeeee Please kindly confirm if the numbers of your PRs during hacktoberfest is right

I worked on 2 PRs 188 and 275

@Mayaleeeee
Copy link
Member

Yeahh
Mine is 4 @AceTheCreator

@quetzalliwrites quetzalliwrites added the Hacktoberfest Label issues as available for participants of https://hacktoberfest.digitalocean.com label Dec 2, 2022
AceTheCreator and others added 13 commits December 5, 2022 16:21
Co-authored-by: Alejandra Quetzalli  <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli  <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli  <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli  <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli <alejandra.olvera.novack@gmail.com>
Co-authored-by: Lukasz Gornicki <lpgornicki@gmail.com>
Co-authored-by: akshatnema <20bcs022@iiitdmj.ac.in>
Co-authored-by: Akshat Nema <76521428+akshatnema@users.noreply.github.com>
Co-authored-by: Akshat Nema <76521428+akshatnema@users.noreply.github.com>
@AceTheCreator
Copy link
Member Author

Updated the PR cc @alequetzalli @derberg

Copy link
Member

@quetzalliwrites quetzalliwrites left a comment

Choose a reason for hiding this comment

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

❤️

@derberg derberg requested a review from akshatnema December 12, 2022 12:25
Co-authored-by: Lukasz Gornicki <lpgornicki@gmail.com>
Copy link
Member

@derberg derberg left a comment

Choose a reason for hiding this comment

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

we merge it tomorrow

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.

approved 👍

@akshatnema
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit 61377cc into asyncapi:master Dec 13, 2022
@derberg
Copy link
Member

derberg commented Dec 13, 2022

/rtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest Label issues as available for participants of https://hacktoberfest.digitalocean.com ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.