👷 Deploy request for celo-composer pending review.

Visit the deploys page to approve it

Hey @Philix27 can you please also add Sveltekit code in this PR as well instead of linking it to your repo?
Sorry for late reply.

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

I have moved the template to this repository. @viral-sangani

Socket Security is pointing to sveltekit core packages (sveltejs/kit, esbuild) needed to run the app. I feel we can let some of these packages pass through.

Can we remove the use of template url and just clone a subdirectory from the packages folder in this repo. @viral-sangani

When the cli runs and Minipay is selected, an external repo is cloned instead of copying from the list of packages/app we have here.

@viral-sangani

Any luck with reviewing this PR?

@Philix27 It does make sense to clone the sub directory rather than cloning from different URL, but we are trying to keep all the templates outside the celo composer repo and all the code components i.e. Basic React + Hardhat etc in the main repo.
How about this -

• I have create a new repo - https://github.com/celo-org/svelte-template, if you move your current svelte code to this repo, and use this as clone URL that would be best.
• Also remove the svelte code from here.

Initially we planned to keep the code here and give an option between ReactJS and Svelte, but looking at the requirement for svelte vs react in web3 development, we want to keep react as default options and svelte as a template which is maintained by community.

@viral-sangani Okay, I tried pushing to the new repo but my access was denied. I can't even make a PR.

@Philix27 Can you try now? You can fork the repo and then add your code and create PR to upstream.

@viral-sangani Okay, the Sveltekit template has been removed and a PR created.