Frontmatter
| id | 8455 |
| title | Remove obsolete path substring in Release Component |
| state | Closed |
| labels | airefactoring |
| assignees | tobiu |
| createdAt | Jan 9, 2026, 10:35 AM |
| updatedAt | Jan 9, 2026, 10:38 AM |
| githubUrl | https://github.com/neomjs/neo/issues/8455 |
| author | tobiu |
| commentsCount | 1 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Jan 9, 2026, 10:38 AM |
Remove obsolete path substring in Release Component
tobiu assigned to @tobiu on Jan 9, 2026, 10:35 AM

tobiu
Jan 9, 2026, 10:37 AM
Input from Gemini 3 Pro:
✦ Successfully updated
getContentPathinapps/portal/view/news/release/Component.mjs.Changes:
- Refactored the method signature to destructure
{path}.- Removed the
.substring(1)call to support the new relative path format.- Updated the JSDoc to reflect the destructured parameter.
/** * @param {Object} record * @param {String} record.path * @returns {String|null} */ getContentPath({path}) { return path ? Neo.config.basePath + path : null }
tobiu closed this issue on Jan 9, 2026, 10:38 AM
In
apps/portal/view/news/release/Component.mjs, thegetContentPath()method currently removes the first character of the path usingsubstring(1).This was necessary when paths started with
/(e.g.,/.github/...). Now that release note paths are relative (e.g.,resources/content/...), this logic is obsolete and breaks the path resolution.Task: Refactor
getContentPathto appendrecord.pathdirectly toNeo.config.basePath.