Add cross env and test windows build

[preschian]

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.

PR type

• Bugfix
• Feature
• Refactoring

What's new?

• can't build with latest main #3594 (comment)
  • add cross-env to support running dev on windows
  • test build with windows environments in github actions

Before submitting Pull Request, please make sure:

• My contribution builds clean without any errors or warnings
• I've merged the recent default branch -- main and I've no conflicts
• I've tried to respect high code quality standards
• I've didn't break any original functionality
• I've posted a screenshot of demonstrated change in this PR

Optional

• I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
• I've tested PR on mobile and everything seems works
• I found edge cases
• I've written some unit tests 🧪

Had issue bounty label?

• Fill up your KSM address: Payout

Community participation

• Are you at KodaDot Discord?

Screenshot

• My fix has changed something on UI; a screenshot is best to understand changes for others.

[netlify]

✅ Deploy Preview for koda-nuxt ready!

Name	Link
🔨 Latest commit	e84611f
🔍 Latest deploy log	https://app.netlify.com/sites/koda-nuxt/deploys/62eba35ffbbd450008839c27
😎 Deploy Preview	https://deploy-preview-3603--koda-nuxt.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.

[preschian]

cc @petersopko

[DeluneClaire]

I'm not sure I understand what should i test; it seems to me as a code only.

[petersopko]

I'm not sure I understand what should i test; it seems to me as a code only.

don't worry about this one, it's code only :)

[petersopko]

cc @petersopko

let's wait for paritytech/smoldot#2565 to test this out and see it working :)

[preschian]

let's wait for paritytech/smoldot#2565 to test this out and see it working :)

seems already released. because the package is too deep, need parent dependencies to update also. we can speed it up to test by forcing the package version with overrides to force the package version.

"overrides": {
  "@substrate/smoldot-light": "^0.6.27"
}

before	after

[petersopko]

let's wait for paritytech/smoldot#2565 to test this out and see it working :)

seems already released. because the package is too deep, need parent dependencies to update also. we can speed it up to test by forcing the package version with overrides to force the package version.

"overrides": {
  "@substrate/smoldot-light": "^0.6.27"
}

hmm, just adding this "overrides" thing doesn't help, I've added it to package.json, hit pnpm i and then tried build again, it still gets stuck at 0.6.25. Any idea how to achieve what you've got there on the right side @preschian ?

[preschian]

hmm, just adding this "overrides" thing doesn't help, I've added it to package.json, hit pnpm i and then tried build again, it still gets stuck at 0.6.25. Any idea how to achieve what you've got there on the right side @preschian ?

hmm, just add this line, like in this PR. with that changes in this PR, the build was a success in windows
refer to this file, bottom line section https://github.com/kodadot/nft-gallery/blob/1dd8ba7817adc4dc24aeb6320b17d48b61128c6f/package.json

[petersopko]

hmm, just adding this "overrides" thing doesn't help, I've added it to package.json, hit pnpm i and then tried build again, it still gets stuck at 0.6.25. Any idea how to achieve what you've got there on the right side @preschian ?

hmm, just add this line, like in this PR. with that changes in this PR, the build was a success in windows refer to this file, bottom line section https://github.com/kodadot/nft-gallery/blob/1dd8ba7817adc4dc24aeb6320b17d48b61128c6f/package.json

this actually worked! wonderful 🚀

let's wait for @prachi00 to check the code and we're good to go

[petersopko]

wfm

[prachi00]

LGTM

[petersopko]

@preschian I'm wondering whether we have to have the over ride here now, since the packages and dependencies were updated? polkadot-js/api@8308715

[preschian]

@preschian I'm wondering whether we have to have the over ride here now, since the packages and dependencies were updated? polkadot-js/api@8308715

still not yet I think, latest version was released 4 days ago, and the PR just merged yesterday

oh, one thing. need to update the GitHub branch protection rule, because in this PR I added build for windows, and steps names were automatically renamed to build (ubuntu-latest) and build (windows-latest) from build

something like this:
cc @yangwao @vikiival

[vikiival]

[petersopko]

pay 100 usd

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $100 @ 60.85 USD/KSM ~ 1.643 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0x049fccc01e29739105279a34a909add93fb6b70af0b08c5a336aaf7a03e13f19

🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues