Toàn
Toàn
That was a quick reply thinkker3197. Sorry that I kind of take it my way and add so many things. If you think any changes are unnecessary, please feel free...
@dallington that `extend` function that cause the error is the original one that I extracted and move to bottom. I checked and it only used once when initializing the settings....
Hi @benmccann, thanks for you insight. I was a bit rush when creating this PR so didn't have time to properly explain the issue, sorry. ### The issue My use...
> Thanks for submitting this, but cloudflare already allows you to deploy with git commits as specified here: https://sonicjs.com/deploy Yeah, this PR just add another way to make a deployment,...
It worked in the CI by using the wrangler config generated from environment variables. For local environment, we have to do the same: 1. Add `.env` file with correct environment...
Sure, I will update it later today. And I can confirm that this will have no breaking change (it will skip the config generation if the env is not set)
> https://github.com/lane711/sonicjs-public/blob/main/src/pages/deploy.mdx Oop, just checked and I can't seem to able to access this repo (or maybe typo?), could you double check @lane711 ?