-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor/cleanup #172
Refactor/cleanup #172
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/vercel/hyper-site/q7ss2b8ul |
refactor header
fix random padding on ios safari, toggle now flips
📝Changed routes:
And 2 other routes: 4 deleted routes:
Commit 63b1aaf (https://hyper-site-git-fork-vassbence-master.vercel.now.sh). |
"husky": { | ||
"hooks": { | ||
"pre-commit": "lint-staged" | ||
} | ||
}, | ||
"lint-staged": { | ||
"*.js": [ | ||
"prettier --write", | ||
"git add" | ||
] | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason to remove this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Husky did not work with [id] named pages, it was a known bug, I can re add it in a sec, if they have since fixed it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's a config you can use:
// lint-staged.config.js
const escape = require('shell-quote').quote
module.exports = {
'**/*.{js,ts,tsx}': filenames => {
const escapedFileNames = filenames
.map(filename => `"${escape([filename])}"`)
.join(' ')
return [
`eslint --max-warnings 0 ${filenames.join(' ')}`,
`prettier --write ${escapedFileNames}`,
`git add ${escapedFileNames}`
]
}
}
pages/store/[name]/source.js
Outdated
|
||
const formatFileName = (path) => path.replace(/^\/+|\/+$/g, '') | ||
|
||
export default () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add SSG to this page too 🙏
The source code won't change that much for plugins, and you can add unstable_revalidate
to SSG, the page will be revalidated in the background after X seconds
Feature/ssg plugin source
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GREAT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing!
This will fix #171.