Skip to content
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.

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

[BUG] u256> display for non-zero u256 #344

Open
metalboyrick opened this issue Nov 6, 2024 · 19 comments
Open

[BUG] u256> display for non-zero u256 #344

metalboyrick opened this issue Nov 6, 2024 · 19 comments
Assignees
Labels

Comments

@metalboyrick
Copy link
Collaborator

metalboyrick commented Nov 6, 2024

Issue Overview

u256> display for non-zero u256, This will happen to all non-zero type-wrappers. We expect only u256 will appear on these cases

image

Proposed Solutions or Ideas

  • Parse generic type and extract the insides.
@metalboyrick metalboyrick added bug Something isn't working difficulty: medium labels Nov 6, 2024
@metalboyrick metalboyrick self-assigned this Nov 6, 2024
@metalboyrick metalboyrick removed their assignment Dec 12, 2024
@metalboyrick metalboyrick changed the title u256> display for non-zero u256 [BUG] u256> display for non-zero u256 Dec 12, 2024
@jedstroke
Copy link

May I take this issue on?

@Supa-mega
Copy link

Is this issue still available?

@emarc99
Copy link

emarc99 commented Dec 12, 2024

Mind if I take this issue? I will do this in 2 days max. Completed basecamp, Nodeguardian quest and starklings, also contributed to OD in Cairo

@1nonlypiece
Copy link

Hi,
I’d like to work on this issue as I have experience with Cairo and building on StarkNet, including writing and optimizing smart contracts.

ETA:
I estimate completing this task in about 12-24 hours. Let me know if I can get started!

@Jagadeeshftw
Copy link

Hi,
I’d love to work on this issue as I have strong experience in developing secure and efficient smart contracts using Cairo and deploying them on StarkNet. Please assign this to me, and I’ll ensure to deliver a robust and high-quality PR within 12-24 hours.

@greatest0falltime
Copy link

Hi,

I’d be thrilled to work on this issue as I have extensive experience with Cairo programming and building decentralized applications on StarkNet.

ETA:
I expect to have a working solution ready within the next 24 hours. Kindly let me know if I can proceed!

Looking forward to contributing!

@caxtonacollins
Copy link

Is it okay if I tackle this? this will be my first time contributing.

@FR3DD221
Copy link

Could I take a shot at this?

@ryzen-xp
Copy link

Hi, I am a proficient blockchain developer, expert in Cairo, Rust, Solidity, Aptos, and web development. I have contributed to various projects like Chainevents and Arkprojects etc.
I will implement a solution to parse the generic type and correctly extract the inner value for non-zero u256 instances, ensuring that only u256 appears as expected. Please assign this issue to me.

Thank you!

@ShantelPeters
Copy link
Contributor

I'd love to work on this!

@Amarjeet325
Copy link

Can I take care of this issue?

@vestor-dev
Copy link

hey sir i would love to work on this issue
i'm a blockchain dev and i would love to handle this issue
assign me and i'll get to work

@metalboyrick
Copy link
Collaborator Author

hey @jedstroke , any updates on the issue?

@jedstroke
Copy link

@metalboyrick I am making a PR soon

@Nadai2010
Copy link
Collaborator

Hi @jedstroke looking forward to your PR, remember that the ODHack will finish on Sunday.

@metalboyrick
Copy link
Collaborator Author

@jedstroke gm, do let us know if you have any problems ser!

@jedstroke
Copy link

@metalboyrick there isn't any problems, honestly. I will make the PR soon trust me 🙏🏽

@metalboyrick
Copy link
Collaborator Author

Sure, please keep in mind that you need some time window for PR reviews too

@Nadai2010
Copy link
Collaborator

Last day to upload your PR @jedstroke, thank you

@jedstroke
Copy link

Last day to upload your PR @jedstroke, thank you

I am done boss @Nadai2010. I am pushing the PR fix shortly. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.