Custom next.config.js support for admin-ui#9105
Custom next.config.js support for admin-ui#9105ggpwnkthx wants to merge 1 commit intokeystonejs:mainfrom
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.jsexists, it will get merged into keystone's admin-ui's NextJS config at runtime.