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

Add Borsh/Json -Schema to types #1158

Merged
merged 29 commits into from
Mar 28, 2024
Merged

Add Borsh/Json -Schema to types #1158

merged 29 commits into from
Mar 28, 2024

Conversation

PolyProgrammist
Copy link
Contributor

@PolyProgrammist PolyProgrammist commented Mar 15, 2024 •
edited
Loading

Adds ability to set borsh attributes as #[near(serializers=[borsh(use_discriminant=false), json])
Adds near macro which contain borsh/json schema for all the types

@PolyProgrammist
add nearschema wherever possible
7f94d65
@PolyProgrammist
tmp
26cc62e
@PolyProgrammist
using nearschema
52a3dfe
@PolyProgrammist
fix cfg
b36c3c4
@PolyProgrammist
use get_schema_derive inside NearSchema
8fccbd6
@PolyProgrammist
tmp
09d1d7d
@PolyProgrammist
Fix problems with TokenMetadata, LookupMap
5887f59
@PolyProgrammist
Try switch to near macro for CurveType enum
97f250b
@PolyProgrammist
tmp
5a1d1a4
@PolyProgrammist
tmp
4278ae0
@PolyProgrammist
Previously added serializers=[borsh(use_discriminant = true)], now re…
2fd4874 
…moved use_borsh_crate=false
@PolyProgrammist
refactor a bit
24a7ae0
@PolyProgrammist
switch from borshserialize to near macro
4a924e2
@PolyProgrammist
fix clippy and fmt
6e1666c
@PolyProgrammist
update clap
680a7fb
@PolyProgrammist
tmp
1402f8f
@PolyProgrammist
tmp
3c5213c
@PolyProgrammist
allow dead_code
ac6c311
@PolyProgrammist
refactor
a2e9dd3
@PolyProgrammist
ci
1668551
@PolyProgrammist
add tests
32f5750
@PolyProgrammist
refactor: add incrementally borsh & json in near macro
f985461
@PolyProgrammist
add cfg_attr inside schema_derive
e0ab667
@PolyProgrammist
refactor: incremental schema_derive
56781ce
@PolyProgrammist
refactor: near_bindgen
a35390c
@PolyProgrammist
fmt
877cbce
@PolyProgrammist
refactor: remove prints
f7bb6b1
@PolyProgrammist
fix compilation test
10d6846
@PolyProgrammist
fmt
70669e6
@PolyProgrammist PolyProgrammist marked this pull request as ready for review March 28, 2024 16:20
frol
frol approved these changes Mar 28, 2024
View reviewed changes
Copy link
Collaborator

@frol frol left a comment

Choose a reason for hiding this comment

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

@PolyProgrammist Awesome job!

@frol frol merged commit c87582f into near:master Mar 28, 2024
14 checks passed
@frol frol mentioned this pull request Mar 28, 2024
Merged
@frol frol mentioned this pull request Apr 25, 2024
Closed
@frol frol mentioned this pull request Jun 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frol frol frol approved these changes

@agostbiro agostbiro Awaiting requested review from agostbiro

@uint uint Awaiting requested review from uint

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PolyProgrammist @frol