Skip to content

WIP: VWBL Gatway Ver2 - #86

Open
thashimoto1998 wants to merge 44 commits into
masterfrom
feat/gatewayv2
Open

WIP: VWBL Gatway Ver2#86
thashimoto1998 wants to merge 44 commits into
masterfrom
feat/gatewayv2

Conversation

@thashimoto1998

Copy link
Copy Markdown
Contributor

No description provided.

@seki-seki seki-seki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

一点コメントしましたが、この実装方針でいいと思います

* Their allocation numerators are deleted, effectively removing them from the active validator set.
* @param newValidators An array of validator addresses that represents the new active validator set.
*/
function unregisterLeaveValidators(address[] memory newValidators) private {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

引数の変数名おかしくない?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

確かにです。leaveValidatorsに修正します。

@seki-seki

Copy link
Copy Markdown
Contributor

あ、fiatnameからfiatIndexが引けるようにしといたほうがいいな

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