Skip to content

Gas optimization in NFT raffler #19

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Gas optimization in NFT raffler #19

wants to merge 2 commits into from

Conversation

@ProgramCrafter
Copy link

@ProgramCrafter ProgramCrafter commented Jan 16, 2023

Creating cell and parsing it takes at least 600 gas units, so it's better to avoid it where cell data is constant.

@Bobish0101
Copy link

Bobish0101 commented Sep 3, 2023

Jjj

JanKortes212
JanKortes212 reviewed Aug 11, 2024
View reviewed changes
Copy link

@JanKortes212 JanKortes212 left a comment

Choose a reason for hiding this comment

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

ok

@Aminzarbani
Copy link

Aminzarbani commented Nov 20, 2024

بله درست است

@amin1419
Copy link

amin1419 commented Dec 25, 2024

run

@Lowthug
Copy link

Lowthug commented Jan 8, 2025

Creating cell and parsing it takes at least 600 gas units, so it's better to avoid it where cell data is constant.

@supervipo
Copy link

supervipo commented Jan 9, 2025

>_

@Parandparmedek1395
Copy link

Parandparmedek1395 commented Jan 30, 2025

Weregood

@Parandparmedek1395
Copy link

Parandparmedek1395 commented Jan 30, 2025

aminzarmani6

@Lowthug Lowthug mentioned this pull request Feb 11, 2025
Open
@kulih-kyber kulih-kyber mentioned this pull request Jul 20, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

31 more reviewers

@Makar21ivan Makar21ivan Makar21ivan approved these changes

@Parandparmedek1395 Parandparmedek1395 Parandparmedek1395 left review comments

@Amin649136411 Amin649136411 Amin649136411 left review comments

@Aminzarbani Aminzarbani Aminzarbani approved these changes

@beh71 beh71 beh71 approved these changes

@Armss9936 Armss9936 Armss9936 approved these changes

@10208612 10208612 10208612 approved these changes

@devilish66694 devilish66694 devilish66694 approved these changes

@DPLE1612 DPLE1612 DPLE1612 approved these changes

@Aaaalex777 Aaaalex777 Aaaalex777 approved these changes

@supervipo supervipo supervipo approved these changes

@Samansonbol80 Samansonbol80 Samansonbol80 approved these changes

@Badevertime Badevertime Badevertime approved these changes

@MrG9090 MrG9090 MrG9090 approved these changes

@JanKortes212 JanKortes212 JanKortes212 approved these changes

@chel1103 chel1103 chel1103 approved these changes

@Olalekan58 Olalekan58 Olalekan58 approved these changes

@tofan42 tofan42 tofan42 approved these changes

@pedylanto pedylanto pedylanto approved these changes

@B1106834 B1106834 B1106834 approved these changes

@johndpierre1 johndpierre1 johndpierre1 approved these changes

@apelsin19921412 apelsin19921412 apelsin19921412 approved these changes

@Njabe91 Njabe91 Njabe91 approved these changes

@nicccc938 nicccc938 nicccc938 approved these changes

@Jubaxrd Jubaxrd Jubaxrd approved these changes

@suroosh18 suroosh18 suroosh18 approved these changes

@markoschalaalx markoschalaalx markoschalaalx approved these changes

@shahshahii shahshahii shahshahii approved these changes

@OniKyoko OniKyoko OniKyoko approved these changes

@Ahmitub Ahmitub Ahmitub approved these changes

@Bobish0101 Bobish0101 Bobish0101 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.