gatsby-theme-carbon icon indicating copy to clipboard operation
gatsby-theme-carbon copied to clipboard

Gatsby v5 ☂️

Open eng618 opened this issue 1 year ago • 4 comments

Summary

Upgrade the theme to use Gatsby v5

Justification

Keeping up with technology changes, and security compliance.

### Tasks
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1425
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1426
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1427
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1428
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1439
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1440
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1449
- [ ] https://github.com/carbon-design-system/gatsby-theme-carbon/issues/1429

eng618 avatar Aug 04 '23 19:08 eng618

Most of this work is done on the next branch.

The remaining pieces are as follows:

eng618 avatar Aug 04 '23 19:08 eng618

For a little more context. The defaultLayouts node for the gatsby-plugin-mdx plugin is no longer a valid configuration. So, the below templates for Default.js and Homepage.js need to be converted into components that can be injected into every page. There are a few ways to do this, as outlined in the linked docs above.

{
  resolve: `gatsby-plugin-mdx`,
  options: {
    extensions: mdxExtensions,
    gatsbyRemarkPlugins: [
      ...defaultRemarkPlugins,
      ...gatsbyRemarkPlugins,
    ],
    remarkPlugins,
    defaultLayouts: {
      default: require.resolve('./src/templates/Default.js'),
      home: require.resolve('./src/templates/Homepage.js'),
      ...extraLayouts,
    },
  },
},

eng618 avatar Sep 01 '23 19:09 eng618

More observed issues in next -

  • [x] Tables listing the component props is not being rendered correctly
  • [x] Code blocks having more than 8 lines are not being rendered correctly
Screenshot 2024-02-08 at 2 06 16 AM Screenshot 2024-02-08 at 2 06 32 AM

muditjuneja avatar Feb 07 '24 20:02 muditjuneja

@eng618 was looking at the webpack warnings and I think it may be something we can ignore. https://robertmarshall.dev/blog/fix-warn-chunk-commons-mini-css-extract-plugin-error-in-gatsby-js/

alisonjoseph avatar Mar 13 '24 18:03 alisonjoseph