Closed surchs closed 6 months ago
Name | Link |
---|---|
Latest commit | 24278e3c7a98eaf66f320f2d22c27629b7be1f63 |
Latest deploy log | https://app.netlify.com/sites/neurobagel-documentation/deploys/663bda73c886e5000837c85b |
Deploy Preview | https://deploy-preview-189--neurobagel-documentation.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Alright folks, thanks a lot for the in depth review here - I appreciate the effort. I went through the comments / suggestions and addressed most of them. The purpose of this PR (and the new docs section) to me is really to get a sysadmin from first contact to running stack in as few steps as possible - and to provide additional detail after that to go in depth. As long as we check this box, I think this PR has served its purpose. Some of this will have to go in later issues / PRs.
There were also a couple of change requests that are in scope of the PR, but that I disagree with in the interest of keeping the "getting started" section very action oriented and simple. I added my thoughts inline, but for me it all comes down to making the first contact with a new deploying user as easy, clear, and successful as possible so we can then tell them the details when they are excited. That said, we will continue refining the style with user feedback and make sure we get it right.
Please both take another look at and let me know if there are any final changes needed. Let's wrap this up and spread the good word!
Changes proposed in this pull request:
configuration
: background section with details on the different configuration options you need to know about.maintenance
: contains the sections on how to handle data updates. Added:Probably best to review this in the staging deploy because the diff is pretty sizable but text mainly has just moved around.
Checklist
Please leave checkboxes empty for PR reviewers
[ENH]
,[FIX]
,[REF]
,[TST]
,[CI]
,[MNT]
,[INF]
) see our Contributing Guidelines for more info)Closes #XXXX