-
Notifications
You must be signed in to change notification settings - Fork 0
Add reference document link to JSON model view #115
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…m:mongoosejs/studio into codex/add-view-document-link-on-json-view
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds the ability to show clickable reference links for ObjectId fields in the JSON document view. When hovering over a referenced ObjectId value, a "View Document" link appears that navigates to the referenced document.
Key changes:
- Updated mongoose devDependency from 8.x to 9.x
- Added reference mapping that tracks which fields reference other models
- Enhanced filterDocument to preserve nested document structure using mpath
- Implemented hover-based reference link UI in the JSON viewer
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated mongoose devDependency to version 9.x |
| frontend/src/models/models.js | Added referenceMap computed property and improved filterDocument to preserve document structure |
| frontend/src/models/models.html | Passed referenceMap to list-json component |
| frontend/src/list-json/list-json.js | Added references prop and implemented reference detection/navigation logic |
| frontend/src/list-json/list-json.html | Passed references prop to json-node component |
| frontend/src/list-json/json-node.html | Extracted template to separate file with reference link UI |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
Testing
Codex Task