-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Custom next.config.js support for admin-ui #9105
base: main
Are you sure you want to change the base?
Conversation
Maybe it would be only write to Users can then do whatever they want? |
when #9186 is merged, user should be able to do this |
OK, this pull request is superseded by #9186 |
@dcousens |
Yes, but I don't have time to test this particular pull request before then - can you? |
sure, I can help regularly, do you want to create a label for me which I can keep eye on? I mean create a label like 'pr help wanted' and i can jump in, I usually do not miss any PR from my GitHub notification except occassionally. Or maybe just assign to me and I can revert back with my finding on specific PR. |
I tried testing this, I had to install lodash as dependency which is missing in PR. I see that there is workaround available to allow using a custom nextjs config. Instead of releasing this I would wait for #9186 this is how to achieve same thing without needing this PR.
export default config({
// ....other config
ui: {
getAdditionalFiles: [
() => [{ mode: 'copy', inputPath: path.resolve('next.config.js'), outputPath: 'next.config.js' }]
],
}
}) |
If
/.keystone/admin/../../next.config.js
exists, it will get merged into keystone's admin-ui's NextJS config at runtime.