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

[Refactor] Move tutorials from Blog to Tutorial section #103

Merged
merged 4 commits into from
Aug 25, 2022

Conversation

rajarshimaitra
Copy link
Contributor

This is a move only PR and moves all the tutorial materials from blog posts to tutorial segment.

I believe we have enough tutorials right now to make sense to keep them separate from regular blog post sections..

I made the move of the files but the webpage is not showing the articles in the tutorial segment..

@netlify
Copy link

netlify bot commented Jun 10, 2022

Deploy Preview for awesome-golick-685c88 ready!

Name Link
🔨 Latest commit dc26c13
🔍 Latest deploy log https://app.netlify.com/sites/awesome-golick-685c88/deploys/630377c549671b0009a439c1
😎 Deploy Preview https://deploy-preview-103--awesome-golick-685c88.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@rajarshimaitra
Copy link
Contributor Author

@ConorOkus @notmandatory Does this require some more changes to correctly build??

@ConorOkus
Copy link
Collaborator

Hmm try running npm install locally and committing again.

@ConorOkus
Copy link
Collaborator

@rajarshimaitra rajarshimaitra force-pushed the tutorial-move branch 2 times, most recently from ea14c04 to c1b7bb4 Compare June 11, 2022 14:38
@rajarshimaitra
Copy link
Contributor Author

Remember to update the links as well

Yup.. That's what I missed.. Fixed..

@rajarshimaitra
Copy link
Contributor Author

  • Removed the old hello world blog.
  • Moved the Descriptor based paper wallet creation demo into tutorial from blog..

rajarshimaitra added a commit to rajarshimaitra/bitcoindevkit.org that referenced this pull request Jun 11, 2022
After bitcoindevkit#103 the tutorial title comes in the side bar in the website,
its better to have shorter names.
@rajarshimaitra
Copy link
Contributor Author

Added title to the Hello World piece..

Also removed the permalink field. Seemed they were pointing to previous locations and not used anymore?? Let me know if thats not ok..

@rajarshimaitra
Copy link
Contributor Author

@rajarshimaitra I mean page heading, these start with a paragraphs

Ahh I missed those.. Added in the last push.. Also made the heading sizes consistent across tutorials..

rajarshimaitra added a commit to rajarshimaitra/bitcoindevkit.org that referenced this pull request Jul 19, 2022
After bitcoindevkit#103 the tutorial title comes in the side bar in the website,
its better to have shorter names.
@rajarshimaitra
Copy link
Contributor Author

Added changes of #105 into this PR.

It seems they aren't used anymore and pointing to previous locations.
This adds `## Introduction` heading to all tutorial articles and
makes heading sizes consistent across all tutorials..
After bitcoindevkit#103 the tutorial title comes in the side bar in the website,
its better to have shorter names.
@rajarshimaitra
Copy link
Contributor Author

Rebased on master..

@notmandatory notmandatory merged commit dc26c13 into bitcoindevkit:master Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants