Skip to content
This repository has been archived by the owner on Aug 14, 2023. It is now read-only.

CallData Decode API #181

Merged
merged 31 commits into from Oct 22, 2020
Merged

CallData Decode API #181

merged 31 commits into from Oct 22, 2020

Conversation

YaronWittenstein
Copy link
Contributor

@YaronWittenstein YaronWittenstein commented Oct 18, 2020 •
edited

Motivation

Adding ergonomic API for decoding the CallData (will work against the Decoder and Cursor).

@YaronWittenstein
WIP
6398942
@YaronWittenstein
* Raw Storage API
ef78f1a 
* Implemented `getters` & `setters` for primitives
@YaronWittenstein
storage-proc-macro: testing bool-type fields.
34b36d9
@YaronWittenstein
storage-proc-macro: test for Amount.
7c3af92
@YaronWittenstein
storage-proc-macro: adding tests for `u8 / i8 / u16 / i16 / u32 / i32…
1c529ea 
… / u64 / i64`
@YaronWittenstein
adding a test for AddressOwned.
207d2ca
@YaronWittenstein
WIP: adding getters & setters of Array-typed fields.
941fb12
@YaronWittenstein
proc-macro storage tests: renaming
12f5aca
@YaronWittenstein
extracting repetitive expression to a macro.
e562d3f
@YaronWittenstein
refactoring
b0c5711
@YaronWittenstein
more refactoring to AppStorage proc-macro.
237027c
@YaronWittenstein
WIP: getters & setters for Address / AddressOwned
2afa2e2
@YaronWittenstein
testing Array-type fields (except for Address and AddressOwned).
8c56f02
@YaronWittenstein
WIP: making Address more ergonomic.
d567643
@YaronWittenstein
deleting storage directory from svm-sdk-macros (code has moved to…
95fb54e 
… `svm-sdk`)
@YaronWittenstein
Simplifying Address - removing AddressOwned and staying with `Add…
6ba63b2 
…ress` (a thin wrapper around `*const u8`).
@YaronWittenstein
get/set for Address array works.
bf2be98
@YaronWittenstein
tests pass.
0ab8bbe
@YaronWittenstein
Changing to proc-macro-attribute.
8343653
@YaronWittenstein
remove unused code.
2f4f428
@YaronWittenstein
extracting storage parsing logic to storage.rs
4ac0046
@YaronWittenstein
WIP
29b4c59
@YaronWittenstein
WIP
72688a2
@YaronWittenstein YaronWittenstein marked this pull request as ready for review October 19, 2020 10:47
@YaronWittenstein YaronWittenstein added the enhancement New feature or request label Oct 19, 2020
@YaronWittenstein YaronWittenstein self-assigned this Oct 19, 2020
@YaronWittenstein YaronWittenstein changed the title Calldata API CallData API Oct 19, 2020
@YaronWittenstein YaronWittenstein changed the title CallData API CallData Decode API Oct 19, 2020
@YaronWittenstein YaronWittenstein mentioned this pull request Oct 21, 2020
Merged
@YaronWittenstein YaronWittenstein merged commit eeca3b1 into master Oct 22, 2020
@YaronWittenstein YaronWittenstein deleted the calldata-api branch October 22, 2020 07:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@moshababo moshababo moshababo approved these changes

Assignees

@YaronWittenstein YaronWittenstein

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@YaronWittenstein @moshababo