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.

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

[New Net P1] - change timestamps from using bigint to using UInt #1842

Open
GrandinLuc opened this issue Feb 20, 2025 · 0 comments · Fixed by #1869
Open

[New Net P1] - change timestamps from using bigint to using UInt #1842

GrandinLuc opened this issue Feb 20, 2025 · 0 comments · Fixed by #1869
Assignees
@GrandinLuc
Milestone
2.0.0-beta (Pre-R...

Comments

@GrandinLuc
Copy link
Collaborator

Summary

VCDM UInts are better suited for this than bigint

Basic Example

timestamp: bigint

becomes

timestamp: UInt
@GrandinLuc GrandinLuc added this to the 2.0.0-beta (Pre-Release) milestone Feb 20, 2025
@GrandinLuc GrandinLuc self-assigned this Feb 20, 2025
@GrandinLuc GrandinLuc changed the title 💡 [REQUEST] - change timestamps from using bigint to using UInt [New Net P1] - change timestamps from using bigint to using UInt Feb 20, 2025
@GrandinLuc GrandinLuc mentioned this issue Feb 26, 2025
Merged
10 tasks
@GrandinLuc GrandinLuc linked a pull request Feb 26, 2025 that will close this issue
1842 new net p1 change timestamps from using bigint to using uint #1869
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GrandinLuc GrandinLuc

Labels
None yet
Projects
None yet
Milestone
2.0.0-beta (Pre-Release)
Development

Successfully merging a pull request may close this issue.

1842 new net p1 change timestamps from using bigint to using uint vechain/vechain-sdk-js
1 participant
@GrandinLuc