Skip to content

Fix requiredETH calculation in buyWithETH function to account for decimal scaling #31

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Copilot wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix requiredETH calculation in buyWithETH function to account for decimal scaling #31

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
319a54e
Initial plan
Copilot Sep 1, 2025
bec2c76
Initial plan for fixing requiredETH calculation
Copilot Sep 1, 2025
685ea95
Fix requiredETH calculation in buyWithETH function
Copilot Sep 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions contracts/SEQICO.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ contract SEQICO is Ownable {
uint256 _pricePerTokenETH,
uint256 _pricePerTokenUSDT,
uint256 _pricePerTokenUSDC
) {
) Ownable(msg.sender) {
seqToken = IERC20(_seqToken);
usdt = IERC20(_usdt);
usdc = IERC20(_usdc);
Expand All @@ -37,7 +37,7 @@ contract SEQICO is Ownable {

function buyWithETH(uint256 tokenAmount) external payable {
require(tokenAmount > 0, "Amount must be greater than 0");
uint256 requiredETH = pricePerTokenETH * tokenAmount;
uint256 requiredETH = pricePerTokenETH * tokenAmount / 1e18;
require(msg.value >= requiredETH, "Insufficient ETH sent");
require(seqToken.balanceOf(address(this)) >= tokenAmount, "Not enough SEQ tokens");

Expand Down
2 changes: 1 addition & 1 deletion contracts/SEQToken.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ contract SEQToken is ERC20, Ownable {
uint256 totalSupply,

@sonnyquinn24 sonnyquinn24 Sep 8, 2025

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

'Total supply is 1,750,000

address owner,
address icoContract
) ERC20("SEQ Token", "SEQ") {
) ERC20("SEQ Token", "SEQ") Ownable(owner) {
// Mint 10% to owner, 90% to ICO contract
uint256 ownerAmount = (totalSupply * 10) / 100;
uint256 icoAmount = totalSupply - ownerAmount;
Expand Down
34 changes: 2 additions & 32 deletions hardhat.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,34 +1,4 @@
require("@nomicfoundation/hardhat-toolbox");

/** @type import('hardhat/config').HardhatUserConfig */
const config = {
module.exports = {
solidity: "0.8.24",
networks: {
hardhat: {},
sepolia: {
url: process.env.SEPOLIA_RPC_URL || `https://sepolia.infura.io/v3/${process.env.INFURA_API_KEY}`,
accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
},
mainnet: {
url: process.env.MAINNET_RPC_URL || `https://mainnet.infura.io/v3/${process.env.INFURA_API_KEY}`,
accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
},
polygon: {
url: process.env.POLYGON_RPC_URL || `https://polygon-mainnet.infura.io/v3/${process.env.INFURA_API_KEY}`,
accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
},
bsc: {
url: process.env.BSC_RPC_URL || "https://bsc-dataseed1.binance.org",
accounts: process.env.PRIVATE_KEY ? [process.env.PRIVATE_KEY] : [],
},
},
gasReporter: {
enabled: process.env.REPORT_GAS !== undefined,
currency: "USD",
},
etherscan: {
apiKey: process.env.ETHERSCAN_API_KEY,
},
};

module.exports = config;
};
Loading
Loading