search

jaibhedia / ChainzRegistry

NFT Based Land Records Management
https://noplaybutton-41656.web.app/
MIT License
9 stars 20 forks source link

in solidity contracts #20

Open kairveeehh opened 1 month ago

kairveeehh commented 1 month ago

in the land contract file there is a error which can lead to gas limit issue , in the MyLands, MyReceivedLandRequest, and MySentLandRequest are storing arrays of values. If these arrays grow too large, it could lead to gas limit issues.we can instead use mapping of structs

please assign under gssoc'24 @jaibhedia

nupur06p commented 1 month ago

Could you please upload the contract file screenshot to check the issue.

kairveeehh commented 1 month ago

Could you please upload the contract file screenshot to check the issue.

um are you the project manager?

nupur06p commented 1 month ago

I was just trying to help by looking at the code. Anyway, lets leave it.

jaibhedia commented 1 month ago

Hey! @kairveeehh you have been assigned the issue, please start working on it.