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

analogWrite.adoc #467

Closed
wants to merge 1 commit into from
Closed

analogWrite.adoc #467

wants to merge 1 commit into from

Conversation

charvi-077
Copy link

No description provided.

@robsoncouto
Copy link
Contributor

Hey @smellai, these pull requests (#465 and #464 as well) have links to youtube videos. I am guessing these are not allowed (since almost all current links reference only other pages of the reference or arduino.cc tutorials). Also, I think that to merge this kind of link into the documentation one would have to watch the videos thoroughly (one of them is 31 minutes, for example).

The README has nothing on this, although one of the samples shows how to embed a video into the page.

Could you confirm or assign someone, please?
Thanks!

@smellai
Copy link
Contributor

smellai commented Oct 22, 2018

@cm7799, we appreciate the work you've been doing for the Arduino community related to the Documentation. However, you included some links to video tutorials on YouTube. While they may be useful, our policy is not to allow links to external resources that we can't control or maintain. Therefore, we cannot approve this pull request. We hope you won't be too disappointed by this, though, and will continue to contribute to the Documentation.

@smellai smellai closed this Oct 22, 2018
@charvi-077
Copy link
Author

charvi-077 commented Oct 22, 2018 via email

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.

3 participants