fix: website revamp #458
fix: website revamp #458
Conversation
Signed-off-by: Lakshya Satpal <[email protected]>
github-actions
bot
added
area/docs
|
@LakshyaSatpal have you confirmed the gatsby version? |
|
we are on Gatsby v5.12.4, I see, still the issue persists. |
|
Running build... |
Signed-off-by: Lakshya Satpal <[email protected]>
There was a problem hiding this comment.
Build warning:
warn `isModuleDeclaration` has been deprecated, please migrate to `isImportOrExportDeclaration`
at isModuleDeclaration
(/Users/lee/code/sistent/site/node_modules/@babel/types/lib/validators/generated/index.js:2748:35)
at PluginPass.Program (/Users/lee/code/sistent/site/node_modules/babel-plugin-lodash/lib/index.js:102:44)
There was a problem hiding this comment.
➜ site git:(pr/LakshyaSatpal/458) make build
gatsby build && gatsby serve
⠋ compile gatsby files
/bin/sh: line 1: 54230 Segmentation fault: 11 gatsby build
make: *** [build] Error 139
Here's what I'm using to build:
gatsby -v
Gatsby CLI version: 5.3.1
Gatsby version: 5.12.9
Note: this is the Gatsby version for the site at: /Users/lee/code/sistent/site
Details:
success Writing page-data.json and slice-data.json files to public directory - 0.111s - 28/29 261.85/s
naviation
ERROR UNKNOWN
Truncated page data information for the failed page "/identity/typography/overview/": {
"errors": {},
"path": "/identity/typography/overview/",
"slicesMap": {},
"pageContext": {}
}
failed Building static HTML for pages - 3.872s
ERROR #95313 HTML.COMPILATION
Building static HTML failed for path "/identity/typography/overview/"
See our docs page for more info on this error: https://gatsby.dev/debug-html
649 | rest = _objectWithoutPropertiesLoose(_ref7, _excluded);
650 | let {
> 651 | basename
| ^
652 | } = React.useContext(UNSAFE_NavigationContext);
653 | // Rendered into <a href> for absolute URLs
654 | let absoluteHref;
WebpackError: TypeError: Cannot destructure property 'basename' of 'react__WEBPACK_IMPORTED_MODULE_0__.useContext(...)
' as it is null.
- index.js:651
[sistent]/[react-router-dom]/dist/index.js:651:1
- router.js:48
[sistent]/[@remix-run]/router/dist/router.js:48:1
- router.js:53
[sistent]/[@remix-run]/router/dist/router.js:53:26
- router.js:56
[sistent]/[@remix-run]/router/dist/router.js:56:1
- router.js:50
[sistent]/[@remix-run]/router/dist/router.js:50:1
- router.js:56
[sistent]/[@remix-run]/router/dist/router.js:56:1
- router.js:59
[sistent]/[@remix-run]/router/dist/router.js:59:1
- router.js:58
[sistent]/[@remix-run]/router/dist/router.js:58:1
- router.js:56
[sistent]/[@remix-run]/router/dist/router.js:56:1
- router.js:59
[sistent]/[@remix-run]/router/dist/router.js:59:1
- router.js:51
[sistent]/[@remix-run]/router/dist/router.js:51:1
- router.js:56
[sistent]/[@remix-run]/router/dist/router.js:56:1
- router.js:59
[sistent]/[@remix-run]/router/dist/router.js:59:1
- router.js:58
[sistent]/[@remix-run]/router/dist/router.js:58:1
- router.js:56
[sistent]/[@remix-run]/router/dist/router.js:56:1
- router.js:51
[sistent]/[@remix-run]/router/dist/router.js:51:1
make: *** [build] Error 1
Signed-off-by: Lakshya Satpal <[email protected]>
|
react-router-dom's routing features were probably clashing with gatsby, and we don't need them both. Removed react-router-dom and build should run now. |
|
LGTM i don't see any build errors |
|
yes the site is building properly @LakshyaSatpal good work. I wonder there quite few pages or components in there why they are been not shown , is there any routing did you discover @LakshyaSatpal ? |
|
@sudhanshutech yes, actually pages of site are still incomplete and need some work. This is not due to my changes in the PR. |
|
@LakshyaSatpal thanks for this. @iArchitSharma @sudhanshutech I'm not sure that it's worth publishing in its current state. |
There was a problem hiding this comment.
@iArchitSharma and @sudhanshutech this branch builds for you? I have pulled the latest commits and get this:
➜ site git:(pr/LakshyaSatpal/458) make setup
npm install
removed 3 packages, and audited 1363 packages in 4s
252 packages are looking for funding
run `npm fund` for details
9 moderate severity vulnerabilities
To address issues that do not require attention, run:
npm audit fix
To address all issues (including breaking changes), run:
npm audit fix --force
Run `npm audit` for details.
➜ site git:(pr/LakshyaSatpal/458) make site
gatsby develop
⠙ compile gatsby files
make: *** [site] Segmentation fault: 11
➜ site git:(pr/LakshyaSatpal/458) give
➜ site git:(pr/LakshyaSatpal/458) make build
gatsby build && gatsby serve
⠙ compile gatsby files
/bin/sh: line 1: 60907 Segmentation fault: 11 gatsby build
make: *** [build] Error 139
Yes it does. success compile gatsby files - 14.195s
success load gatsby config - 0.064s
success load plugins - 0.385s
success onPreInit - 0.002s
success delete worker cache from previous builds - 0.012s
success initialize cache - 0.079s
success copy gatsby files - 0.489s
success Compiling Gatsby Functions - 0.543s
success onPreBootstrap - 0.570s
success createSchemaCustomization - 0.002s
success Clean up stale nodes - 0.043s - 28/0 0.00/s
success Checking for changed pages - 0.001s
success source and transform nodes - 0.142s
info Writing GraphQL type definitions to /home/shivam/layer5/sistent/site/.cache/schema.gql
success building schema - 0.609s
success createPages - 0.015s
success createPagesStatefully - 0.428s
info Total nodes: 50, SitePage nodes: 28 (use --verbose for breakdown)
success Checking for changed pages - 0.002s
success Cleaning up stale page-data - 0.008s
success onPreExtractQueries - 0.001s
success extract queries from components - 0.272s
success write out redirect data - 0.002s
success onPostBootstrap - 0.001s
info bootstrap finished - 37.641s
success write out requires - 0.037s
success Building production JavaScript and CSS bundles - 2.588s
success Building HTML renderer - 4.955s
success Execute page configs - 0.112s
success Caching Webpack compilations - 0.001s
success run queries in workers - 0.009s - 0/0 0.00/s
success Merge worker state - 0.003s
success Rewriting compilation hashes - 0.002s
naviation
naviation
naviation
success Building static HTML for pages - 131.620s - 28/28 0.21/s
info There are no new or changed slice html files to build.
success stitching slices - 0.184s
success onPostBuild - 0.002s
info Done building in 178.145494373 sec
Pages
┌ src/pages/404.js
│ ├ /404/
│ └ /404.html
├ src/pages/index.js
│ └ /
├ src/pages/components/index.js
│ └ /components/
├ src/pages/home/index.js
│ └ /home/
├ src/pages/identity/Content.js
│ └ /identity/Content/
├ src/pages/identity/Heading.js
│ └ /identity/Heading/
├ src/pages/identity/Navigation.js
│ └ /identity/Navigation/
├ src/pages/identity/Next.js
│ └ /identity/Next/
├ src/pages/identity/Previous.js
│ └ /identity/Previous/
├ src/pages/identity/Sidebar.js
│ └ /identity/Sidebar/
├ src/pages/identity/SubContent.js
│ └ /identity/SubContent/
├ src/pages/identity/SubHeading.js
│ └ /identity/SubHeading/
├ src/pages/identity/SubText.js
│ └ /identity/SubText/
├ src/pages/identity/Text.js
│ └ /identity/Text/
├ src/pages/patterns/index.js
│ └ /patterns/
├ src/pages/visualise/index.js
│ └ /visualise/
├ src/pages/identity/color/code.js
│ └ /identity/color/code/
├ src/pages/identity/color/guidance.js
│ └ /identity/color/guidance/
├ src/pages/identity/color/index.js
│ └ /identity/color/
├ src/pages/identity/color/overview.js
│ └ /identity/color/overview/
├ src/pages/identity/elevation/index.js
│ └ /identity/elevation/
├ src/pages/identity/page-layouts/index.js
│ └ /identity/page-layouts/
├ src/pages/identity/spacing/index.js
│ └ /identity/spacing/
├ src/pages/identity/typography/code.js
│ └ /identity/typography/code/
├ src/pages/identity/typography/guidance.js
│ └ /identity/typography/guidance/
├ src/pages/identity/typography/index.js
│ └ /identity/typography/
└ src/pages/identity/typography/overview.js
└ /identity/typography/overview/
╭────────────────────────────────────────────────────────────────╮
│ │
│ (SSG) Generated at build time │
│ D (DSG) Deferred static generation - page generated at runtime │
│ ∞ (SSR) Server-side renders at runtime (uses getServerData) │
│ λ (Function) Gatsby function │
│ │
╰────────────────────────────────────────────────────────────────╯
Something is already running at port 9000
⠀
You can now view sistent in the browser.
⠀
http://localhost:9002/ |
Yeah i agree, the current state is just raw format: |
|
My suggestion is make to the above page as comming soon or under development type message(commenting out the footer and unnecessary body) including in this pr so we can move forward with the @LakshyaSatpal changes so that after we can start building pages. Make sense @leecalcote @nebula-aac ? In this way this will not be hanging // @LakshyaSatpal @ritiksaxena124 @Rexford74 |
Signed-off-by: Lakshya Satpal <[email protected]>
Signed-off-by: Lakshya Satpal <[email protected]>
Signed-off-by: Lakshya Satpal <[email protected]>
Signed-off-by: Lakshya Satpal <[email protected]>
|
Hi @LakshyaSatpal , You're welcome at the meeting Check to community calendar for more details of meetings at Layers. |
|
@LakshyaSatpal what is the status here, i pinged you in slack too. Are you working on this , what the plan here. |
|
merge conflict |
|
@lakshz what would you like to do with this one (this PR) going forward? |
|
@leecalcote I'm unsure, if the work done in this PR is usable further or not. I am adding a screenshot of the website in this PR. The problem which was stated before, stays, that how do we make the design components reusable from existing layer5 or meshery sites?
I'm happy to work on both of the above steps. |
|
Good suggestions, @lakshz. I'd like to add that some of the current assets are being improved in the process, from buttons to modals and so on. So even when components are being reused, which is a major part of what we are aiming to achieve with Sistent, these improvements should be accounted for. |
|
Sounds good. Once the reusable assets are more clear, I'll try to have something by next Monday. |
|
Cool! I'll be in touch, @lakshz. |
|
Merge conflicts @lakshz |
Signed-off-by: lakshya <[email protected]>
|
@ritiksaxena124 Resolved. |
Notes for Reviewers
This PR fixes site build error.
Errors was majorly due to omitting React import in components, gatsby doesn't allow it. gatsbyjs/gatsby#28657
There were some issues because of react-router-dom v6 was used, fixed that.
Signed commits