Skip to content

Conversation

@josh-padnick
Copy link
Contributor

Add a page mentioning repo-copier as a way to self-host the library

@netlify
Copy link

netlify bot commented Jun 16, 2023

Deploy Preview for pensive-meitner-faaeee ready!

Name Link
🔨 Latest commit 41e1a96
🔍 Latest deploy log https://app.netlify.com/sites/pensive-meitner-faaeee/deploys/6490c05051c58f000824ef87
😎 Deploy Preview https://deploy-preview-897--pensive-meitner-faaeee.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.

Copy link
Contributor

@oredavids oredavids left a comment

Choose a reason for hiding this comment

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

Content looks good. Left one minor suggestion.

The tests don't pass currently due to the docs/guides/stay-up-to-date/index.md changes. Please revert that. We unfortunately haven't worked on preventing that from happening when working locally.

@oredavids
Copy link
Contributor

Did not catch this earlier but the self-hosting markdown file needs to be moved into the _docs-sources directory and then run the yarn regenerate:local command so it is copied over to docs with its metadata.

@josh-padnick
Copy link
Contributor Author

josh-padnick commented Jun 19, 2023

@oredavids I'm sorry, but I'm afraid I'm confused here. I keep trying variations of moving the file and running yarn regenerate:local. But It doesn't work. Would you mind taking a pass at what I need to do to get this fixed?

@oredavids
Copy link
Contributor

@josh-padnick It appears that your last commit fixed the issues so i did not have to do anything.

@josh-padnick
Copy link
Contributor Author

Ah nice, thank you! Thought I'd already tried that, but guess not. Merging now!

@josh-padnick josh-padnick merged commit cd4c223 into master Jun 19, 2023
@josh-padnick josh-padnick deleted the add-repo-copier-page branch June 19, 2023 21:01
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