feat: Add initial data management pages #62
Conversation
✅ Deploy Preview for calypr ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This PR updates the CALYPR data management documentation to reflect a significant tooling migration from Gen3/forge to git-drs/GRIP. The changes modernize the documentation with current command syntax, expand workflow guidance, and add comprehensive examples for working with FHIR metadata and graph-based queries.
Changes:
- Rebranded commands from
forgeand Gen3 SDK togit-drsand GRIP throughout all documentation - Added comprehensive workflow guides for data management including file addition, metadata generation, validation, and querying
- Updated the index page with expanded FHIR data model overview and CALYPR tool ecosystem description
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/calypr/data-management/metadata.md | Updated validation commands from forge meta validate to git-drs meta validate |
| docs/calypr/data-management/managing-metadata.md | Rebranded validation tools from forge to git-drs; contains one copy-paste error in command example |
| docs/calypr/data-management/index.md | Complete rewrite from "FHIR for Researchers" to comprehensive CALYPR Data Management guide with tool ecosystem overview |
| docs/calypr/data-management/git-drs.md | Expanded from basic instructions to comprehensive workflow guide with detailed examples |
| docs/calypr/data-management/data-model/metadata.md | Updated from gen3_util commands to git-drs and GRIP commands for metadata management |
| docs/calypr/data-management/data-model/integration.md | Updated reference link and added review date footer |
| docs/calypr/data-management/custom-views.md | Added review date footer with minor formatting issue (duplicate separator) |
| docs/calypr/data-management/common-errors.md | Added review date footer |
| docs/calypr/data-management/analysis/query.md | Complete rewrite from Gen3 SDK/GraphQL to GRIP/GRIPQL query examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| --- No newline at end of file | ||
| --- | ||
| --- |
There was a problem hiding this comment.
There are two consecutive horizontal rule markers (---). This appears to be a duplicate - only one separator is needed before the "Last reviewed" footer.
| --- |
| **Check-edge-** Example: | ||
|
|
||
| \`\`\`forge check-edge META\`\`\` or \`\`\`forge validate META/DocumentReference.ndjson\`\`\` | ||
| \`\`\`git-drs meta check-edge META\`\`\` or \`\`\`git-drs meta validate META/DocumentReference.ndjson\`\`\` |
There was a problem hiding this comment.
The second example command should use check-edge instead of validate to match the section title and first example. The command should be git-drs meta check-edge META/DocumentReference.ndjson.
There was a problem hiding this comment.
I think this is incorrect. git-drs just manages the files, it doesn't know about edge generation. That is something that forge is in charge of
lbeckman314
force-pushed
the
feat/data-management
branch
from
3e7810e to
f632da0
Compare
lbeckman314
force-pushed
the
feat/data-management
branch
from
0a92c3f to
3d6689f
Compare
Overview
This PR addresses Issue #60 by adding initial data management pages.
Tip
Preview → https://deploy-preview-62--calypr.netlify.app/calypr/data-management/