Skip to content

Create a list of local speakers#1993

Merged
ZYSzys merged 4 commits intonodejs:masterfrom
ladyleet:patch-2
Jan 23, 2019
Merged

Create a list of local speakers#1993
ZYSzys merged 4 commits intonodejs:masterfrom
ladyleet:patch-2

Conversation

@ladyleet
Copy link
Copy Markdown
Contributor

Create a list of local speakers for local meetups to connect with! :)

Create a list of local speakers for local meetups to connect with! :)
@ladyleet
Copy link
Copy Markdown
Contributor Author

@obensource @dshaw @bnb @joesepi @mhdawson hayyyy! :) inspired after the user feedback meeting to create this too.

@ladyleet
Copy link
Copy Markdown
Contributor Author

@chowdhurian here you can poke at this one too :)

Copy link
Copy Markdown
Contributor

@keywordnew keywordnew left a comment

Choose a reason for hiding this comment

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

LGTM! Solid idea for a resource 🎉

@ladyleet
Copy link
Copy Markdown
Contributor Author

@chowdhurian gotta get it merged so you can add yourself!

Co-Authored-By: ladyleet <tracy@thisdot.co>
Copy link
Copy Markdown
Contributor

@willin willin left a comment

Choose a reason for hiding this comment

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

LGTM.

btw, i wonder why only Canada using emoji

@SofiEstevez
Copy link
Copy Markdown
Contributor

@mhdawson I think we are ready to merge right?

Co-Authored-By: ladyleet <tracy@thisdot.co>
@ladyleet
Copy link
Copy Markdown
Contributor Author

@chowdhurian so after we created these two we need to submit PRs to update the actual site so they show up as resources no?

keywordnew added a commit to keywordnew/nodejs.org that referenced this pull request Jan 23, 2019
@keywordnew
Copy link
Copy Markdown
Contributor

keywordnew commented Jan 23, 2019

@ladyleet from what I understood of @ZYSzys' reference, locale/en/site.json needs an update. #1999 should sort that.

@ZYSzys ZYSzys merged commit ae61495 into nodejs:master Jan 23, 2019
@ZYSzys
Copy link
Copy Markdown
Member

ZYSzys commented Jan 23, 2019

Thanks :)

ZYSzys pushed a commit that referenced this pull request Jan 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants