Skip to content

First draft of ThunderHub chapter for myNodeDocs - #20

Merged
abhiShandy merged 8 commits into
mynodebtc:masterfrom
davterra:ThunderHub
Sep 24, 2020
Merged

First draft of ThunderHub chapter for myNodeDocs#20
abhiShandy merged 8 commits into
mynodebtc:masterfrom
davterra:ThunderHub

Conversation

@davterra

Copy link
Copy Markdown
Contributor

Ok, first draft of ThunderHub chapter.

Please review & critique for content & length.

Comment thread docs/lightning/thunderhub.md Outdated
Comment thread docs/lightning/thunderhub.md
Comment thread docs/lightning/thunderhub.md Outdated
Fix incorrect image
Add image
Add info about thunderhub port and file location

@davterra davterra left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@abhiShandy
I applied your suggestions and pushed a fresh version.
Let me know of any more changes that you think are needed.

Comment thread docs/lightning/thunderhub.md Outdated
Comment thread docs/lightning/thunderhub.md Outdated
Comment thread docs/lightning/thunderhub.md Outdated
Comment thread docs/lightning/thunderhub.md Outdated
Comment thread docs/lightning/thunderhub.md Outdated
@abhiShandy

Copy link
Copy Markdown
Collaborator

Looks good to me

@abhiShandy

abhiShandy commented Sep 21, 2020

Copy link
Copy Markdown
Collaborator

I pulled this branch to visualize it locally.

I have few suggestions:

  1. Center the thumbnail for thunderhub (check the bitcoin/manage.md for reference)
  2. Do you think adding a "Specs" subsection at the end, like shown below, will be helpful?

image

@davterra

Copy link
Copy Markdown
Contributor Author

I pulled this branch to visualize it locally.

I have few suggestions:

  1. Center the thumbnail for thunderhub (check the bitcoin/manage.md for reference)
  2. Do you think adding a "Specs" subsection at the end, like shown below, will be helpful?

image

Yeah I considered centering the thumbnail and decided to wait for your opinion. I'll do that...

I like the "specs" idea. But then we should do that for each app.

@davterra davterra left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@abhiShandy I applied your suggested changes with one exception: In the new "Specs" section at the end, I left out the version #. It seems to me that if we include the version number for every app on its respective page that we'll soon have a full-time job just keeping them all current, as apps get upgraded and Taylor applies them in myNode.
Let me know what you think. I'm going to now start on the RTL page, which is currently not too bad in Taylor's docs, but it needs updating. Let me know if you have higher priorities.
After RTL I thought I'd just start working my way through the "Help Wanted" tags in the Issues.

@abhiShandy

Copy link
Copy Markdown
Collaborator

I agree. We should remove the version from Specs. Let's remember to have this subsection for every app. I'll take a final look and merge it.

You can start working on RTL and other apps.

@abhiShandy
abhiShandy merged commit 93ca6dc into mynodebtc:master Sep 24, 2020
@abhiShandy

Copy link
Copy Markdown
Collaborator

#3

@abhiShandy abhiShandy added this to the 1.0.0 milestone Sep 24, 2020
mynode-dev pushed a commit that referenced this pull request Aug 24, 2026
* First draft of ThunderHub chapter for myNodeDocs

* Fix incorrect image

* Update docs/lightning/thunderhub.md

* minor edits

* Center thumbnail image and add "specs" section at the end.

* reformat few lines, add tile image and cleanup

Co-authored-by: abhishandy <abhishek.sandy@gmail.com>
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.

2 participants