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

Use prefix in serialize/deserialize of serde. #3340

Merged
merged 1 commit into from
Dec 22, 2022

Conversation

AurelienFT
Copy link
Contributor

This fix a problem that the API doesn't respond with the IDs with their prefix and version and so now it use their fromstr and display implementation and so use them. Linked to #3279

@aoudiamoncef aoudiamoncef merged commit 934b594 into testnet_18 Dec 22, 2022
@aoudiamoncef aoudiamoncef deleted the use_prefix_version_serde branch December 22, 2022 14:21
@AurelienFT AurelienFT mentioned this pull request Dec 23, 2022
bors bot added a commit that referenced this pull request Jan 2, 2023
3280: Testnet 18 r=AurelienFT a=AurelienFT

PR for testnet 18. PRS in this testnet : 

- #3303 
- #3301 
- #3279 
- #3318 
- #3315
- #3332
- #3228 
- #3323 
- #3312 
- #3333 
- #3336 
- #3340 
- #3206 
- #3320 
- #3314 
- #3342 
- #3343 
- #3346
- #3339 
- #3351 
- #3349 
- #3354 
- #3348 
- #3347 

Co-authored-by: AurelienFT <aurelien.foucault@epitech.eu>
Co-authored-by: AurelienFT <32803821+AurelienFT@users.noreply.github.com>
Co-authored-by: Thomas Plisson <thomas.plisson@epitech.eu>
Co-authored-by: Eitu33 <89928840+Eitu33@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants