Skip to content

Conversation

dharkmattr
Copy link

No description provided.

Copy link
Contributor

@cwsnt cwsnt left a comment

Choose a reason for hiding this comment

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


uint256 public constant POINTS = 1000;

uint256 public inputFeeInPoints = 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably can just removed the 0 initialization here, since it will use the zero value by default

uint256 public constant POINTS = 1000;

uint256 public inputFeeInPoints = 0;
uint256 public outputFeeInPoints = 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably can just removed the 0 initialization here, since it will use the zero value by default

uint256 public inputFeeInPoints = 0;
uint256 public outputFeeInPoints = 0;

address public feeVaultAddress = address(0);
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably can just removed the 0 initialization here, since it will use the zero value by default

uint256 public outputFeeInPoints = 0;

address public feeVaultAddress = address(0);
address public swapERC20Address = address(0);
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably can just removed the 0 initialization here, since it will use the zero value by default

uint8 v;
bytes32 r;
bytes32 s;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Can add the constructor for the initialization value of the configs

bytes32 s;
}

uint256 public constant POINTS = 1000;
Copy link
Contributor

Choose a reason for hiding this comment

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

By having the points = 1000, this means the smallest fee that we will have is 0.1%, am i correct?

Copy link
Author

Choose a reason for hiding this comment

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

indeed

Choose a reason for hiding this comment

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

Generally POINTS refer to 10,000.

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.

4 participants