chore: upgrade to capi beta40

[peetzweg]

Just upgrading to capi beta40 on it's own to debug the built error with capi 40 in the #186 pr.

[netlify]

✅ Deploy Preview for capi-multisig ready!

Name	Link
🔨 Latest commit	6434c53
🔍 Latest deploy log	https://app.netlify.com/sites/capi-multisig/deploys/6479a9c18f19860008c51a87
😎 Deploy Preview	https://deploy-preview-197--capi-multisig.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[peetzweg]

@kratico cc @ryanleecode
Getting these errors on the console in the deployed version. App is not loading.

I'm a bit confused why the errors are different to the error we had in #186 . In #186 we weren't able to build for production, here it works but results in the error above in the deployment. 🤔
The changes regarding to the capi upgrade look the same to me.

[ryanleecode]

Did you also commit the updates to your vite config?

[ryanleecode]

Did you also commit the updates to your vite config?

[kratico]

The package.json hasn't been updated

capi-multisig-app/package.json

Line 15 in d861039

"capi": "0.1.0-beta.38",

[peetzweg]

Ah yes you both are right @kratico @ryanleecode . Just pushed the actual upgrade to beta40. Which resulted in the same error as in #186, therefore I also adapted the vite.config.ts.

https://deploy-preview-197--capi-multisig.netlify.app/

[ryanleecode]

Were the rollup options in vite config there before? What happens when they are removed

[peetzweg]

Were the rollup options in vite config there before? What happens when they are removed

No they weren't. Without them the pnpm build step fails:

[peetzweg]

Okay seems the combination of these two changes are enough to make it work. I've added a comment that this should be removed once paritytech/capi#1038 is closed.

capi-multisig-app/www/index.html

Lines 18 to 22 in 6434c53

	<script>
	// TODO remove once closed https://github.com/paritytech/capi/issues/1038
	// temporary capi workaround
	window.process = { env: {} };
	</script>

capi-multisig-app/www/vite.config.ts

Lines 16 to 21 in 6434c53

	define: {
	// TODO remove once closed https://github.com/paritytech/capi/issues/1038
	// temporary capi workaround
	"process.env.CAPI_SERVER": process.env.CAPI_SERVER,
	"process.env.CAPI_TARGET": process.env.CAPI_TARGET,
	},