henkaku-center / henkaku-nengajo-contract

MIT License

7 stars 10 forks source link

Feature/calc total registered nengajo #31

Closed yu23ki14 closed 2 years ago

yu23ki14 commented 2 years ago

やったこと

registerCreative の関数名をregisterNengajo に変更
registerNengajo のHENKAKUトークンの計算式を仮組み（トータル枚数で決定するように）。テストの記述
HENKAKUトークンの変化を厳密にみたいので、テストコードをRegisterNengajoとMintNengajoでdescribeを分けた。

33

yawn-c111 commented 2 years ago

@yu23ki14 近々の懸案事項を実装してくださり、ありがとうございます！

一点だけ、 https://github.com/henkaku-center/henkaku-nengajo-contract/blob/292675d4a2685a527342db29bd1b82bf86fd9eed/contracts/Nengajo.sol#L120

https://github.com/henkaku-center/henkaku-nengajo-contract/blob/292675d4a2685a527342db29bd1b82bf86fd9eed/contracts/Nengajo.sol#L172

この２つの関数が重複した機能かなと思います！ご確認お願いします！

yu23ki14 commented 2 years ago

@yawn-c111 あ、ほんとですねあざます！