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

Patch noding section of the tutorial #1745

Merged
merged 2 commits into from
Apr 11, 2019

Conversation

evgenykochetkov
Copy link
Contributor

No description provided.

nkrkv
nkrkv previously requested changes Apr 9, 2019
Copy link
Member

@nkrkv nkrkv left a comment

Choose a reason for hiding this comment

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

On patch naming. Suggest shortening patch names to:

  • 3xx-use-new-node
  • 3xx-document-it

workspace/welcome-to-xod/300-creating-nodes/patch.xodp Outdated Show resolved Hide resolved
workspace/welcome-to-xod/301-create-new-patch/patch.xodp Outdated Show resolved Hide resolved
@evgenykochetkov evgenykochetkov force-pushed the feat-patching-section-for-tutorial branch from a13f6f9 to 4f66fe5 Compare April 9, 2019 14:26
@nkrkv nkrkv changed the title Patching section of the tutorial Patch noding section of the tutorial Apr 9, 2019
@evgenykochetkov evgenykochetkov force-pushed the feat-patching-section-for-tutorial branch 2 times, most recently from bdf2582 to 0afb456 Compare April 10, 2019 10:14
{
"comments": [
{
"content": "# Creating your own nodes\n\nIn this part of the tutorial, you will learn how to create your own XOD nodes.\n\nWe will take an existing patch, and *extract* a part of it into a separate node and add pins to control hard-coded parameters. Then we will add labels and descriptions that will be displayed in the help panel for our node, just like in XOD's standard library.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Camel Cased H1 please in all these tutorials

workspace/welcome-to-xod/305-document-it/patch.xodp Outdated Show resolved Hide resolved
{
"comments": [
{
"content": "# Next steps\n\nWhat you've just learned about making nodes is totally sufficient for day-to-day XODing.\n\nBut if you are interested in a more in-depth guide (for example, if you already have some expirience in Arduino programming), check out https://xod.io/docs/guide/#making-your-own-nodes. There you will learn about creating nodes in C++, testing them using tabtests and wrapping existing Arduino libraries.",
Copy link
Contributor

Choose a reason for hiding this comment

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

UTM, please.

@@ -0,0 +1,43 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably, we should tell somewhere in this tutorial about publishing library (but User should not publish this tutorial). What do you think? :)

{
"comments": [
{
"content": "# Congratulations!\n\nYou have completed the tutorial. Hope you enjoyed it. May we ask to rate it?\n\n[★★★★★ Excellent!](https://forum.xod.io/t/tutorial-feedback/457/?utm_source=ide&utm_medium=ide_comment&utm_campaign=tutorial_99&utm_term=5_stars)\n[★★★★ Very Good](https://forum.xod.io/t/tutorial-feedback/457/?utm_source=ide&utm_medium=ide_comment&utm_campaign=tutorial_99&utm_term=4_stars)\n[★★★ Okay](https://forum.xod.io/t/tutorial-feedback/457/?utm_source=ide&utm_medium=ide_comment&utm_campaign=tutorial_99&utm_term=3_stars)\n[★★ So-so](https://forum.xod.io/t/tutorial-feedback/457/?utm_source=ide&utm_medium=ide_comment&utm_campaign=tutorial_99&utm_term=2_stars)\n[★ Poor](https://forum.xod.io/t/tutorial-feedback/457/?utm_source=ide&utm_medium=ide_comment&utm_campaign=tutorial_99&utm_term=1_star)\n\nIf you would like to give more detailed feedback, we’ll happy to hear it on the rating page.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably, we have to start the new topic to collect comments related only to the new tutorial? 🤔

@evgenykochetkov evgenykochetkov force-pushed the feat-patching-section-for-tutorial branch from 0afb456 to e6a2415 Compare April 11, 2019 11:46
@evgenykochetkov evgenykochetkov force-pushed the feat-patching-section-for-tutorial branch from f38656c to 1429ef2 Compare April 11, 2019 12:01
@brusherru brusherru dismissed nkrkv’s stale review April 11, 2019 13:19

It blocks new tutorial docs.

@brusherru brusherru merged commit 6bf7525 into master Apr 11, 2019
@brusherru brusherru deleted the feat-patching-section-for-tutorial branch April 11, 2019 13:20
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.

None yet

4 participants