Hey guys,
I'm new to solidity and I was looking at your code to try to understand how things work.
Anyway, isn't this test incorrect:

if(_startingTimestamp >= getCurrentTime()) {
     lotteryStatus = Status.Open;
} else {
      lotteryStatus = Status.NotStarted;
}

If the starting timestamp is bigger than the current time, then the status should be NotStarted. It should be like this:

if(_startingTimestamp >= getCurrentTime()) {
     lotteryStatus = Status.NotStarted;
} else {
      lotteryStatus = Status.Open;
}

And maybe I would add a "require" to make sure that _closingTimestamp is bigger than getCurrentTime(). Because there is no point to create a lottery that is already closed.

Can you add the option to buy fractions of lottery tickets?

After buying some tickets, I've noticed that one number of a ticket was repeated.
After checking this file: Lottery.sol

1. Shouldn't the Buy method check and avoid repeated items on the ticket?
2. Shouldn't the Buy method be called inside MultiBuy?

Thanks.

how to understand the issueIndex mean in lotteryNFT.sol?

The lottery logic currently relies on a ERC721 token to handle NFT tickets.

https://bscscan.com/token/0x5e74094cd416f55179dbd0e45b1a8ed030e396a1

Owing to technical limitations (no batch minting) in the ERC721 standard, it is essential to update the lottery logic with a new LotteryNFT built under the ERC1155 standard.

The purpose of this change is to batch mint tickets, which will result in massive gas reduction for users, and allow greater participation from users.

https://docs.openzeppelin.com/contracts/3.x/erc1155

Hi, I get the following error on running yarn deploy:local.

TypeError: lotteryInstance.init is not a function
    at main (/home/eth/lottery-contract/scripts/deploy.js:85:27)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
error Command failed with exit code 1.

Any suggestions why this is the case?