Skip to content

Conversation

@AlexD10S
Copy link
Contributor

@AlexD10S AlexD10S commented Oct 2, 2025

This PR adds more ink! resources:

  • A tutorial on Ethereum compatibility (helpful for Polkadot Hub with MetaMask/Hardhat/Wagmi/ethers.js)
  • A frontend tutorial for building dApps.
  • It also adds Pop CLI as a recommended tool for ink! development.

@AlexD10S AlexD10S force-pushed the alex/include-more-ink-resources branch from 56889dd to 9c5f8ac Compare October 6, 2025 07:04
@AlexD10S
Copy link
Contributor Author

AlexD10S commented Oct 6, 2025

@midegdugarova the CI was failing with the following error:

* [403] <https://www.npmjs.com/package/create-polkadot-dapp?activeTab=readme> | Rejected status code (this depends on your "accept" configuration): Forbidden

It didn’t seem related to this PR, but I added --exclude "https://www.npmjs.com/package/create-polkadot-dapp*" in the CI workflow to unblock the run.
Let me know if there’s a better approach or if we’d prefer to revert this change.

Copy link
Contributor

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @AlexD10S! Just a minor details

@nhussein11
Copy link
Contributor

nhussein11 commented Oct 6, 2025

@midegdugarova the CI was failing with the following error:

* [403] <https://www.npmjs.com/package/create-polkadot-dapp?activeTab=readme> | Rejected status code (this depends on your "accept" configuration): Forbidden

It didn’t seem related to this PR, but I added --exclude "https://www.npmjs.com/package/create-polkadot-dapp*" in the CI workflow to unblock the run. Let me know if there’s a better approach or if we’d prefer to revert this change.

I'm not sure why the CI fails with that link, but it seems to be a problem we've been dealing with for a while -> #64 (comment)

I believe the best solution would be to create a . lycheeignore file to handle these cases -> https://github.com/lycheeverse/lychee-action?tab=readme-ov-file#excluding-links-from-getting-checked

@AlexD10S AlexD10S force-pushed the alex/include-more-ink-resources branch from dfb41a0 to bd21af7 Compare October 6, 2025 11:22
@AlexD10S
Copy link
Contributor Author

AlexD10S commented Oct 6, 2025

@midegdugarova the CI was failing with the following error:

* [403] <https://www.npmjs.com/package/create-polkadot-dapp?activeTab=readme> | Rejected status code (this depends on your "accept" configuration): Forbidden

It didn’t seem related to this PR, but I added --exclude "https://www.npmjs.com/package/create-polkadot-dapp*" in the CI workflow to unblock the run. Let me know if there’s a better approach or if we’d prefer to revert this change.

I'm not sure why the CI fails with that link, but it seems to be a problem we've been dealing with for a while -> #64 (comment)

I believe the best solution would be to create a . lycheeignore file to handle these cases -> https://github.com/lycheeverse/lychee-action?tab=readme-ov-file#excluding-links-from-getting-checked

Cleaner than the exclude, done in bd21af7

@nhussein11
Copy link
Contributor

nhussein11 commented Oct 6, 2025

@midegdugarova the CI was failing with the following error:

* [403] <https://www.npmjs.com/package/create-polkadot-dapp?activeTab=readme> | Rejected status code (this depends on your "accept" configuration): Forbidden

It didn’t seem related to this PR, but I added --exclude "https://www.npmjs.com/package/create-polkadot-dapp*" in the CI workflow to unblock the run. Let me know if there’s a better approach or if we’d prefer to revert this change.

I'm not sure why the CI fails with that link, but it seems to be a problem we've been dealing with for a while -> #64 (comment)
I believe the best solution would be to create a . lycheeignore file to handle these cases -> https://github.com/lycheeverse/lychee-action?tab=readme-ov-file#excluding-links-from-getting-checked

Cleaner than the exclude, done in bd21af7

Sorry, could you please revert this one + remove the exclude thing from the CI? I already created a PR for this haha. We can merge it with the CI failing since it will be fixed by #70.

@AlexD10S AlexD10S force-pushed the alex/include-more-ink-resources branch from bd21af7 to 6805cec Compare October 6, 2025 12:04
@AlexD10S
Copy link
Contributor Author

AlexD10S commented Oct 6, 2025

Sorry, could you please revert this one + remove the exclude thing from the CI? I already created a PR for this haha. We can merge it with the CI failing since it will be fixed by #70.
@nhussein11 done!

@nhussein11 nhussein11 merged commit c612889 into polkadot-developers:master Oct 6, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants