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

[Site Showcase] Adding Dan Abramov's personal blog #10241

Closed
wants to merge 2 commits into from
Closed

[Site Showcase] Adding Dan Abramov's personal blog #10241

wants to merge 2 commits into from

Conversation

daydream05
Copy link
Contributor

Saw his post on Twitter so i thought it might be great to add his new blog to the showcase

Questions though:

  • What's the rule on posting other people's site?
  • Should we ask Dan for permission first before adding it?

Saw his post on Twitter so i thought it might be great to add his new blog to the showcase

Questions though:
- What's the rule on posting other people's site? 
- Should we ask Dan for permission first before adding it?
@daydream05 daydream05 requested a review from a team December 2, 2018 18:17
@gatsbot
Copy link

gatsbot bot commented Dec 2, 2018

Danger run resulted in 1 warning; to find out more, see the checks page.

Generated by 🚫 dangerJS

@KyleAMathews
Copy link
Contributor

Generally you shouldn't add someone's personal blog without permission as individuals should be able to make that decision. Corporate sites are ok. We can ask him here though — @gaearon — would adding your blog to the gatsby site showcase be ok? https://www.gatsbyjs.org/showcase/

@daydream05
Copy link
Contributor Author

Understood 👍

@LekoArts
Copy link
Contributor

Closing this PR then since he didn't answer to this. If he wants it added at some later point he can open up a new PR 👍

@LekoArts LekoArts closed this Dec 30, 2018
@gaearon
Copy link

gaearon commented Jan 21, 2019

I don't mind

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.

4 participants