unchain-tech / unchain-projects Goto Github PK

提案内容

ETH-NFT-Game Section4 Lesson1のApp.jsにsetIsLoadingを追加する箇所で、
fetchNFTMetadata関数を移し忘れている可能性があります。

変更前：
useEffect(() => {
　const fetchNFTMetadata = async () => {
　.
　.
　.
　setIsLoading(false);
}, [currentAccount]);

変更後：
useEffect(() => {
　const fetchNFTMetadata = async () => {
　.
　.
　.
　setIsLoading(false);
　};
　if (currentAccount) {
　　console.log("CurrentAccount:", currentAccount);
　　fetchNFTMetadata();
　}
}, [currentAccount]);

問題：
下記URLのテキストにおいて、下図のPolygon Mainnetという名前が、おそらく最新のチェーンではMatic Mainnetになってると思います。（少なくとも私のMetamaskでは）

https://unchain-portal.netlify.app/projects/4-polygon-generative-nft/section-3-lesson-2

対策
図とテキストが変わる事がベストですが、括弧書きでMatic mainnetと記載があるだけでも十分かと思います、

ETH-NFT-Makerなどバナー画像がそもそも存在しないプロジェクトがある。
また、section-0なのかREADMEなのかの命名が一貫していない。

修正お願いします 🙏🏽
修正は別ブランチにpushの後PRお願いします

@KatotenMk2 @yukis4san

提案内容

SOLANA NFT DROP
Section0 Lesson1
「本プロジェクトは下記の３つのステップに分かれています。」の下記の部分が抜け落ちていると思われます。

備考

該当部分gyazoで撮りました。

https://gyazo.com/2487e800c296f00577d31d0ed9e893ba

提案内容

すいません。もう1つ見つかりました。
SOLANA ONLINE STORE 最後の部分で文章が重複しています。
上の文のairtableのリンクを押すと関係の無いところへ飛びます。

背景

リンクを押したところ、ページ上部へ飛んだので気がつきました。

備考

VScodeで該当する部分を開けてみましたが、文章は1度しか書かれておらず、原因はわかりませんでした。

該当部分gyazoにて撮りました。
https://gyazo.com/b6c7a046df94f952531d09af26dd7fad

提案内容

Solana-Online-Store Section4
vercelの設定がSolana-NFT-Dropの設定になっております。

FRAMEWORK PRESET :
Create React App → Next.js

ROOT DIRECTORY : 
app → ./

NAME: 
SOLANA_NETWORK → NEXT_PUBLIC_OWNER_PUBLIC_KEY

VALUE: 
devnet → (phantom wallet address)

Why

テキストの可読性を上げるため。
背景がグレーで文字色が薄い黒だと色の系統が被るためメリハリがつきにくく、どうしても可読性高くなりにくいかもです。

What

テキストの背景色・文字色をそれぞれ白・黒にする。(ダークモード検討時は逆でも良いかもです。)
例としてnoteをあげてますが、メリハリがついて読みやすいなって思ってます。

unchain	Note

section2からnpm run startを使いますが、終了のさせ方を知らないといちいちターミナルを消していたので、時間が取られます。npm run start をしたあとのターミナルからctrl + cで終了させれるみたいな記述があったらわかりやすいかもしれません

提案内容

学習コンテンツを多言語対応させたい

docs/[CONTENT_TITLE]/下に ja の他にも en es zh など徐々に増やしていきたい

背景

良質なコンテンツが溜まりつつあるので言語によるアクセスへの障壁を取り除きたい

what's this

• PRのtempleteが欲しい

motivation

• GHに慣れてないユーザが少しでも気楽にPR投げられるようにしたい
• PRを見たときに何がしたいPRかひと目で分かるようにしてreveiewコストを減らす

solution

• PRtemplete作るｗ
• あれもこれも盛り込んだマッチョなテンプレは不要（それ自体がコストになる
• 何なら「何をしたか」と、（あるなら）issueへのリンクだけでも良い

misc

• みんなでワイワイ考えよ

内容

NEAR-Hotel-Booking-dApp Section3 Lesson1

以下のファイルが near-hotel-booking-dapp / frontend に存在しません。

near-api.js
near-config.js

背景

インストールしたCreate NEAR Appのテンプレートがコンテンツ作成時のものと異なっている可能性がございます。

備考

Create NEAR App frontend template :
https://github.com/near/create-near-app/tree/master/templates/frontend

提案内容

NEAR-BikeShare Section2 Lesson2

下記のリンク先が見つかりませんでした。

• NEAR開発でのCargo.tomlの書き方 (https://docs.near.org/sdk/rust/building/basic-build)

提案内容

8-Ganache-Yield-Farm を進めていったところ、教材通りに進めるとエラーになるところが２箇所ありましたので報告いたします。
後続の初心者の方もつまづかれると思いますので、ご確認いただけますと幸いです。

背景

Section2-lesson1の画像の部分について、教材記載の import "./DaiToken.sol"; ではエラーがでました。
import "./MockDaiToken.sol"; にしましたら、先へすすめました。
Section3-lesson3の画像の部分について、console.log(stakingBalance.toString())のところエラーがでました。
console.log(tokenFarmBalance.toString())にしましたら、先へすすめました。

備考

Section2-lesson1参考画像
https://gyazo.com/42b031183d963d23214ed14876c1b48a

Section3-lesson3参考画像
https://gyazo.com/f87ca3a64f63d57f587923ce0e4f7db2

gyazoというアプリを使いました。無料故か、下に広告出てしまいます。

提案内容

画象と文字の間が空いている問題

背景

学習画面を見ていたら、画象と文字が大きく空いている（全ての箇所）ので、お手隙の際に詰めていただけると幸いです。
VisualStudioで見たら、文字と画像の間は空白が無かったので、フロントエンド側で調整いただけるとありがたいです！

備考

例：https://app.unchain.tech/learn/Polygon-Generative-NFT/section-1_lesson-1

提案内容

102-ETH-NFT-Collectionのsection-2-Lesson-3について、
中盤のMyEpicNFT.solのコード内容について、
pragma solidity 0.8.9;
と記載されておりますが、エラーが出ました。

前述のコードを参考に、以下と思います。
pragma solidity ^0.8.9;

細かいですがご確認いただけると幸いです。

備考

https://unchain-portal.netlify.app/projects/102-ETH-NFT-Collection/section-2-Lesson-3

section-4-Lesson-1のMyEpicNFT.solも同様に” ^ ”が抜けているようです。
https://unchain-portal.netlify.app/projects/102-ETH-NFT-Collection/section-4-Lesson-1

提案内容

画像の新規差し込み
docs/ASTAR-SocialFi/ja/section-0/Lesson_2_環境構築をしよう.md
https://github.com/unchain-dev/UNCHAIN-projects/blob/main/docs/ASTAR-SocialFi/ja/section-0/Lesson_2_%E7%92%B0%E5%A2%83%E6%A7%8B%E7%AF%89%E3%82%92%E3%81%97%E3%82%88%E3%81%86.md?plain=1
の
// TODO default image of next.js
を新しい画像と差し替える。

266
-- // TODO default image of next.js
++

279
--
++

画像の番号の割り振りを下記のように変更
従来：
className 変更前の画像 none to 0_2_14.png
className 変更後の画像 0_2_14.png to 0_2_15.png

新 0_2_14.png

新 0_2_15.png

why

• run.jsでテストを書くことに違和感がある

what

• hardhatには（というかtruffleには）unittestを書けるmochaやchaiみたいな子がいるのでそちらを使ってみてはどうだろう？
• pros
  　- debugしやすい
  　- 本来の使い方に近い（と思う
• cons
  　- ユニットテストのお作法を覚えなくちゃいけないのがしんどいかも...

misc

• ジャストアイデアな温度感なので議論希望
• 工数かかるし

repository 内にCSSはないのでISSUEだけ起票しておきます。他の箇所にも影響があるので要調整とは思います。

Solana-dApp　Section 1　Lesson 1

Phantom Walletを設定しよう！

が無くなってしまってます。昨日までは、あったように思います。
（　GitHub 慣れてないので、ここへのコメントでいいのか、このコメントが正しいのか分かりませんが取り急ぎ・・・）

提案内容

PhantomWalletの最初の設定で、Devnetに変更する際、
「設定」→「デベロッパー設定」→「ネットワークの変更」→「DevNet」
のように、「デベロッパー設定」を挟みますので、テキストの確認をお願いします。

Why:
下記リンクにおいて、solidityのバージョンが0.8.0と0.8.4が混在しています。マイナーバージョンなので問題無いとは思いますが。

https://unchain-portal.netlify.app/projects/1-eth-dapp/section-4-lesson-1

What:
その他レッスンでは0.8.4を使われているので、下図２箇所を0.8.0から0.8.4に修正するのが適切かと思います。

提案内容

AVAX-Messenger Section2 Lesson5【 ETHとAVAX 】

下記のリンク先が確認できませんでした。

1ETH = 1AVA というわけではありません。
詳しくはこちら (https://docs.avax.network/specs/coreth-arc20s#nativeassetcall) をご覧ください。

提案内容

ETH-NFT-Game Section3 Lesson5のおまけの箇所でopenseaリンクの誤記が見られました。

変更前：
https://testnets.opensea.io/assets/${gameContract.address}:${tokenId.toNumber()}?tab=details

コンテンツ内では上記のようなフォーマットが採用されていますが、このままですと期待するOpenseaのページに移ることはできませんでした。
おそらく、Rarible rinkeby testnet用のリンクになっていると思います。

変更後：
https://testnets.opensea.io/assets/${gameContract.address}/${tokenId.toNumber()}

gameContract.addressの後ろを「 : 」→ 「 / 」に変更すると正しく表示されました。
ご確認のほどよろしくお願いいたします。

提案内容

'Next lesson' の左右対称にある現状の 'Back to project overview' ボタンを 'Previous lesson' ボタンに変えたらどうか？

背景

一つ前のレッスンに戻るボタンだと直感的に思って押して、目次に飛ばされてしまうということが何度もあった。

備考

ここを変えたい　赤の矢印

提案内容

リンク切れがありました。正しいリンク先が分からなかったのでIssueだけ起票しておきます。

https://github.com/shiftbase-xyz/UNCHAIN-projects/blob/main/docs/3-ETH-NFT-game/ja/section-1/Lesson_4_NFT%E3%82%AD%E3%83%A3%E3%83%A9%E3%82%AF%E3%82%BF%E3%83%BC%E3%82%92%E4%BD%9C%E3%82%8D%E3%81%86.md?plain=1#L3

背景

備考

https://unchain-portal.netlify.app/projects/3-ETH-NFT-game/section-1-Lesson-4

提案内容

Rinkebyネットワークはもうdeprecatedなので学習コンテンツの該当箇所をGörliもしくはSepoliaに更新したい

背景

https://blog.ethereum.org/2022/06/21/testnet-deprecation

備考

chroniumベースのブラウザだと点の位置がずれてしまっている
chrome or brave brawser

こっちがsafari
safariだとちゃんと読み込めている

Lesson-4, ABIファイルを取得するにて、最後のApp.jsのコードを再度コピペする形なので、contractAddressが運営さんが用意した 0x3610145E4c6C801bBf2F926DFd8FDd2cE1103493 に戻っちゃって、みんなそっちに挨拶しているようですw
できれば .env 導入したい

• Why:
  下記リンクにて、下図の通り、”setCurrentAccount”の位置がずれている。

https://unchain-portal.netlify.app/projects/2-eth-nft-collection/section-4-lesson-1

• What:
  上の行にそろえる。

提案内容

NEAR-MulPay Section1 Lesson3

swap.test.ts に書かれていないテストコードが説明の箇所にございます。
そちらのコードを書いてテストしましたが、listUpTokenAddress がswap.solに定義されていないとのことでエラーがでました。
コンテンツ内に記されているテスト結果と一致しませんので、説明の部分を省いてもよいかもしれません。

以下明記されていないコード

it("List up token address with that of symbol", async function () {
      const {
        DaiToken,
        EthToken,
        AoaToken,
        ShibToken,
        SolToken,
        UsdtToken,
        UniToken,
        MaticToken,
        SwapContract,
      } = await loadFixture(deployTokenFixture);
      const DAI = ethers.utils.formatBytes32String("DAI");
      const ETH = ethers.utils.formatBytes32String("ETH");
      const AOA = ethers.utils.formatBytes32String("AOA");
      const SHIB = ethers.utils.formatBytes32String("SHIB");
      const SOL = ethers.utils.formatBytes32String("SOL");
      const USDT = ethers.utils.formatBytes32String("USDT");
      const UNI = ethers.utils.formatBytes32String("UNI");
      const MATIC = ethers.utils.formatBytes32String("MATIC");

      const tokenSymbolList = [DAI, ETH, AOA, SHIB, SOL, USDT, UNI, MATIC];
      const tokenAddressList = [
        DaiToken.address,
        EthToken.address,
        AoaToken.address,
        ShibToken.address,
        SolToken.address,
        UsdtToken.address,
        UniToken.address,
        MaticToken.address,
      ];

      for (let i = 0; i < tokenSymbolList.length; i++) {
        await SwapContract.listUpTokenAddress(
          tokenSymbolList[i],
          tokenAddressList[i]
        );
        const tokenAddress = await SwapContract.returnTokenAddress(
          tokenSymbolList[i]
        );
        const tokenSymbol = ethers.utils.parseBytes32String(tokenSymbolList[i]);
        console.log(`${tokenSymbol} token address :${tokenAddress}`);
      }
    });

提案内容

SOLANA ONLINE STORE のSection3 Lesson1ですが、図の部分のdotenvがリンクが切れていると思われます。
ご確認お願い申し上げます。

背景

クリックすると500エラーページへ飛びます。

備考

該当部分をgyazoで撮りました。

https://gyazo.com/f75308c1b44edaeaca7c12095b43c57d

提案内容

ETH-NFT-Game Section4 Lesson3

.gitignoreファイルに、「.DS_Store」以外記載されておりません。
node_modulesなど不必要なものは省いてあげるほうがより丁寧だと感じました。

リモートにプッシュする際、時間が他のコースよりもかかるので驚く方がいらっしゃるかもしれません。

提案内容

NEAR-MulPay Section2 Lesson2 main.dart

main.dartの全体コードをコンテンツに追加

背景

signin.dartの説明後、続けてmain関数の詳細を確認していますが、
main.dartへ移動する等の説明が見当たらないためわかりにくいと感じました。

加えて、main.dartに使用するimportやパスのコードが書かれていないのでこちらもコンテンツに加える必要があると思います。

こちらの

本プロジェクトは以下のステップに分かれています。 // TODO: ステップを記載

の後に記載がないので追記お願いします。 @yk-saito

提案内容

https://app.unchain.tech/learn/ETH-dApp/section-2_lesson-1/
の「ターミナル上でdApp-starter-projectに移動して下記を実行しましょう。」の下にnpm initの実行を促す文言を追加していただいた方がよりテキストが丁寧になるのではないかと思います。

背景

テキスト通りにnpm installを行うとnpm ERR! enoent This is related to npm not being able to find a file.というエラーが発生します。

備考

https://app.unchain.tech/learn/ETH-dApp/section-2_lesson-1/

https://unchain-portal.netlify.app/projects/2-ETH-NFT-collection/section-2-Lesson-3
Base64.solに関して、 貼り付けて下さいとなっているので、どれだけの人がTypeされたり、コードを確認するか分かりませんが、
Hexコードの部分に関して、小文字もしくは大文字で統一しておいた方が初学者に対しては親切なような気がします。

統一した方が良いという話であれば、PRを送ります。

提案内容

https://github.com/unchain-dev/UNCHAIN-projects/blob/2c73353415c41290ef0b9afad94c2f1f1d6e7e6f/docs/1-ETH-dApp/ja/section-1/Lesson_2_Solidity%E3%81%A7%E3%82%B9%E3%83%9E%E3%83%BC%E3%83%88%E3%82%B3%E3%83%B3%E3%83%88%E3%83%A9%E3%82%AF%E3%83%88%E3%82%92%E4%BD%9C%E6%88%90%E3%81%97%E3%82%88%E3%81%86.md?plain=1#L39-L62

「1-ETH-dApp」の Section 1 Lesson 2 にて、コントラクトのライセンスについて上記のように解説があります。

この SPDX-License-Identifier: UNLICENSED という記述は（そもそもSPDXとしてはinvalidのようですが）いわゆるプロプライエタリなコードを示すのに、npmやSolidityなど一部で使われているようです。（cf: https://opensource.stackexchange.com/a/12412）

Unfortunately, the format SPDX-License-Identifier: UNLICENSED is an invalid SPDX license expression per the SPDX spec, because the identifier UNLICENSED is not part of the SPDX license list. While well-intentioned, Solidity has here caused potentially dangerous confusion.

一方で当プロジェクトはLICENSEファイルこそ無いものの、README.mdに「全てオープンソース（MIT ライセンス）で運用されています」とあり、ドキュメント中のサンプルソースコードについても同様に適用されていると期待できます。

もちろん、チュートリアルを実施している各ユーザーがそれぞれの判断において、元コード及びユーザー独自のコードをプロプライエタリとして(再)宣言することは可能でしょう（MITは厳格なcopyleftと違い、コードのプロプライエタリ化も容認しているはずと記憶しています）。しかし、サンプル中にUNLICENSEDがサンプルソースとセットで書いてあるのは、元コードがそもそもプロプライエタリ、流用不可であると誤認できてしまいます。あるいは誤認ではなく意図的な設定であれば、MITライセンスのプロジェクト中に流用不可なコードが混在していることになり、ライセンスに矛盾が生じています。

解決策としては、ドキュメント中では SPDX-License-Identifier: MIT で宣言し、必要であれば UNLICENSED の説明をするのが良いと思われます。（チュートリアル完了時など）

備考

なお、上記StackExchangeの回答にもありますが、SPDXにある Unlicense は The Unlicense という、ソフトウェアでのパブリックドメインを表明するものです（プロプライエタリと真逆）。もしこちらを意図していたならばそのように改訂する手もありますが、 UNLICENSED / Unlicense で紛らわしいので個人的にはお勧めいたしかねます。

デプロイするプロジェクトはdapp-starter-projectであると文字では記載されているが、画像のプロジェクト名はmy-wave-portalになっている。
これはcontract側のプロジェクト名と被っているので、混乱する人がでるかも

まぁちゃんと文章をみればわかることではあるので、勘違いする人が出てから対応してもいいかもと思いました

Similar issue to #13

In section 1 lesson 5 you run the command: npx hardhat node

It might be worth adding a note saying that you quit with control + c.

提案内容

背景

discordで質問があったので調べて見たら変な気がして
ですが動作確認の方法も一から作るのは難しいので、確認したいです

備考

• https://app.unchain.tech/learn/ETH-dApp/ja/1/5/
• 10338d7#r102414222

• discord
  • https://discord.com/channels/936573458748432405/946064452489469962/1080102407591178271

問題
下記リンクにて、下図の赤丸の箇所のforループ、require内の段落がずれてる。

https://unchain-portal.netlify.app/projects/4-Polygon-Generative-NFT/section-2-Lesson-1

対策
赤丸箇所を一段ずらす。

提案内容

https://github.com/unchain-dev/UNCHAIN-projects/blob/main/public/images/102-ETH-NFT-Collection/README/ETH-NFT-banner-blur.png

@yukis4san こちらtypoです、修正&re-uploadお願いします

背景

備考

提案内容

見本レポジトリを改善したのでコンテンツ内容をそれに伴うものに更新

提案内容

コードの中で不要と思われる行

ASTAR-SocialFi/section-2_lesson-1

postFunction.tsx
同じ if が二重になっている
761 if (output !== undefined && output !== null)
762 if (output !== undefined && output !== null)

FT.tsx
使われていない変数　date
625 export const transfer = async (props: PropsTF) => {
630 const date = new Date();

647 export const distributeReferLikes = async (props: PropsDRL) => {
652 const date = new Date();

背景

備考

コードの中身に関わることなので Issue にしています。
妥当であれば、指示いただければPRします。

https://github.com/unchain-dev/UNCHAIN-projects/blob/main/docs/ASTAR-SocialFi/ja/section-2/Lesson_1_%E3%82%B3%E3%83%B3%E3%83%88%E3%83%A9%E3%82%AF%E3%83%88%E3%81%AE%E9%96%A2%E6%95%B0%E3%82%92%E5%91%BC%E3%81%B3%E5%87%BA%E3%81%9B%E3%82%8B%E3%82%88%E3%81%86%E3%81%AB%E3%81%97%E3%82%88%E3%81%86.md?plain=1

起きている事象

• Contract: コントラクト, 契約
• Balance: バランス, 残高　
  と日本語訳が混在している
  e.g. https://unchain-portal.netlify.app/projects/1-ETH-dApp/section-3-Lesson-2

提案内容

• Contact: コントラクト
• Balance: 残高
  に統一する

FYI: Solidity の日本語ドキュメント

• Contract は コントラクト と訳されている
  • https://solidity-jp.readthedocs.io/ja/latest/contracts.html,
• Balance は バランス 及び 残高 と混在している
  • https://solidity-jp.readthedocs.io/ja/latest/search.html?q=%E3%83%90%E3%83%A9%E3%83%B3%E3%82%B9&check_keywords=yes&area=default#
  • https://solidity-jp.readthedocs.io/ja/latest/search.html?q=%E6%AE%8B%E9%AB%98&check_keywords=yes&area=default

バランスだと日本語だと「均衡」を表す言葉としてもありややこしいので、「残高」の方が直感的に伝わり理解しやすいと思うため

提案内容

背景

備考

NEAR-Hotel-Booking-dApp Section2 Lesson4

下記のリンク先に移ると、Page Not Found が表示されます。

👥 サブアカウントを作り直す
作り直す理由としては、予期せぬエラーを防ぐためです。 そのままデプロイをして、何かしらのメソッドを呼び出した際におそらく以下のようなパニックエラーが起こるでしょう。

Cannot deserialize the contract state.
詳しい説明は[こちら](https://www.near-sdk.io/upgrading/prototyping)を確認してください。

現在はこちらで確認できるかと思います。

提案内容

ETH-NFT-Game Section4 Lesson2
imgurリンクをCIDに変更する際、run.jsだけではなくdeploy.jsにも変更する説明をいれたほうがより丁寧だと感じました。

<img
　src={https://cloudflare-ipfs.com/ipfs/${characterNFT.imageURI}}
　alt={Character ${characterNFT.name}}
/>

Arena/index.jsなどで上記のようにsrcをipfs用リンクに変更しているので、
imgurの画像が表示しないエラーが発生します。

提案内容

SOLANA DAPP の Section2 Lesson1、Anchorをインストールする際の説明文中にあるAnchorのリンクですが、クリックすると404ページとなります。リンクが切れているかもしれません。

備考

該当場所gyazoで撮りました。

https://gyazo.com/6f57638b03c5755ed078e7e32d8dd8b0

提案内容

ETH DAOの

• S3L3
• S3L4

の中身がない

背景

n/a

備考

至急、直接PR送ってください

n/a