Skip to content
Closed
12 changes: 6 additions & 6 deletions gatsby-node.js
Original file line number Diff line number Diff line change
Expand Up @@ -528,7 +528,7 @@ exports.createPages = async ({ actions, graphql, reporter }) => {
try {
createPage({
path: pagePath,
component: require.resolve(componentPath),
component: path.resolve(componentPath),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this change

});
} catch (error) {
console.error(`Error creating page for "${pagePath}":`, error);
Expand Down Expand Up @@ -661,11 +661,11 @@ exports.onCreateNode = ({ node, actions, getNode }) => {
case "kanvas-labs":
slug = `/learn/${collection}/${slugify(node.frontmatter.title)}`;
break;
case "resources":
if (node.frontmatter.published)
slug = `/${collection}/${slugify(
node.frontmatter.category
)}/${slugify(node.frontmatter.title)}`;
case "resources":
if (node.frontmatter.published)
slug = `/${collection}/${slugify(
node.frontmatter.category
)}/${slugify(node.frontmatter.title)}`;
break;
case "members":
if (node.frontmatter.published)
Expand Down
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here also

Binary file not shown.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shteypandey28-hue Could you explain the changes made here?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T he maintainer already flagged them — In an earlier cleanup commit (Fix: remove unrelated news files from FormControl PR), those files were explicitly removed because they don't belong.
They're completely unrelated — docker-kanvas-infoq.webp and index.mdx are a news article about

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@saurabhraghuvanshii Could you please take a look? I don’t think we need to remove that recent news article we added.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shteypandey28-hue please revert this, and open Pr only releted to issue. No need to remove new articles and anything. Your work in this is only to add new sistent components.

This file was deleted.

Loading
Loading