Adding a layout component that will be used for every page to wrap it with a proper head and later use the nav and footer component.
The component can take several meta props so it can be customized as needed per page
Details
Details
- Reviewers
bytesofman - Group Reviewers
Restricted Project - Commits
- rABCf342672a7c44: Add layout component
run it and inspect the head
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| web/e.cash/components/layout.js | ||
|---|---|---|
| 31 ↗ | (On Diff #39414) | Is this a nextjs convention? I know that React is deprecating defaultProps What model are you seeing this in? |
Comment Actions
Good call I didn't realize that was deprecated. Changed them to default values as specified here https://react.dev/learn/passing-props-to-a-component#specifying-a-default-value-for-a-prop
Comment Actions
Ah yes they do need to be absolute paths.
To fix I added the url as an environmental variable in a .env.local file as outlined here https://nextjs.org/docs/basic-features/environment-variables
Figured this would be good as its in more than one place and so it can easily be modified if needed