Repository navigation
Conversation
There was a problem hiding this comment.
Code Review
This pull request restructures the Semantic Knowledge Graph documentation by moving it into a dedicated directory and adding comprehensive documentation for the new Manual Declarations feature. It updates relative links across various files, introduces a new category configuration, and documents several new API actions for manual node and dependency management. Feedback on the changes includes a correction to use US English spelling ("favor" instead of "favour") in the API documentation description for kg_apply_manual_node_merge.
mangglesh
force-pushed
the
docs/kg-manual-declarations
branch
2 times, most recently
from
October 7, 2026 06:17
918609d to
58b01c8
Compare
blue4209211
marked this pull request as draft
October 7, 2026 06:26
7 tasks done
mangglesh
force-pushed
the
docs/kg-manual-declarations
branch
from
October 7, 2026 07:04
58b01c8 to
e8cf3b0
Compare
mangglesh
force-pushed
the
docs/kg-feature-guide
branch
from
October 7, 2026 07:34
d4633a0 to
51dd16f
Compare
mangglesh
force-pushed
the
docs/kg-manual-declarations
branch
from
October 7, 2026 07:36
e8cf3b0 to
d068e7c
Compare
mangglesh
force-pushed
the
docs/kg-manual-declarations
branch
from
October 7, 2026 09:57
d068e7c to
dea80e7
Compare
Knowledge Graph Settings now has one Manual Declarations tab, and nodes and dependencies are imported from a single JSON document instead of two CSVs. - New page features/knowledge-graph/manual-declarations: what can be declared and when a declared node merges with a collected one, node and dependency statuses, declaring one at a time, the JSON format with field tables and an example, import outcomes, permissions and removal. - Link it from the Knowledge Graph overview, Where the Data Comes From, and How NudgeBee Uses the Graph. - API docs: replace kg_import_manual_dependencies with kg_create_manual_declarations and add the kg_*_manual_node* actions that were never listed. Co-Authored-By: Claude Opus 5.5 <[email protected]> Signed-off-by: Mangglesh Dagar <[email protected]>
mangglesh
force-pushed
the
docs/kg-manual-declarations
branch
from
October 8, 2026 05:51
dea80e7 to
f326254
Compare
mangglesh
marked this pull request as ready for review
October 8, 2026 06:26
mangglesh
marked this pull request as draft
October 8, 2026 06:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Documents the Knowledge Graph's Manual Declarations: adding dependencies and resources the collectors cannot see, one at a time or from a single JSON document. The feature ships in nudgebee/nudgebee-enterprise#40384. That PR merges the old Manual Dependencies and Manual Nodes tabs into one, and replaces their two CSV imports with a single JSON import.
kg_import_manual_dependenciesis replaced bykg_create_manual_declarations, and the sevenkg_*_manual_node*actions are added. The rows use the routing table's own descriptions, so the next regeneration produces the same text.Type of change
Checklist
npm run buildlocally and the site builds without errorsnpm run lintlocally and lint passesgit commit -s)Screenshots (if visual changes)
The four new images are in
doc-server/docs/features/img/knowledge-graph-*.png, and the new page uses them. They show example names on a dev tenant, cropped to the dialog or section, with no endpoints or user names visible.Additional notes
|and\in action descriptions, so a{or<in a routing-table comment breaks the MDX build. The new action's comment originally contained one. It now sits in a code span, fixed at the source in #40384.