Commit 5b8e1cf
authored
remove camelcase from api responses (#167)
### TL;DR
Updated JSON field names to follow consistent snake_case convention and added backward compatibility for deprecated fields.
### What changed?
- Renamed `decodedData` to `decoded` in transaction and log responses
- Changed `indexedParams` to `indexed_params` and `nonIndexedParams` to `non_indexed_params`
- Added deprecated versions of the params fields to maintain backward compatibility
- Added swagger documentation tags to mark deprecated fields
### How to test?
1. Make API requests that fetch decoded logs and transactions
2. Verify new snake_case fields contain the expected data
3. Confirm deprecated camelCase fields still work and contain the same data
4. Check swagger documentation reflects the deprecated status of old fields
### Why make this change?
To establish consistent snake_case naming conventions across the API while ensuring existing integrations continue to work through a deprecation period.2 files changed
+16
-12
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
65 | 67 | | |
66 | 68 | | |
67 | 69 | | |
| |||
77 | 79 | | |
78 | 80 | | |
79 | 81 | | |
80 | | - | |
81 | | - | |
| 82 | + | |
| 83 | + | |
82 | 84 | | |
83 | 85 | | |
84 | 86 | | |
85 | 87 | | |
86 | | - | |
| 88 | + | |
87 | 89 | | |
88 | 90 | | |
89 | 91 | | |
| |||
251 | 253 | | |
252 | 254 | | |
253 | 255 | | |
254 | | - | |
255 | | - | |
256 | | - | |
257 | | - | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
258 | 262 | | |
259 | 263 | | |
260 | 264 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
56 | | - | |
| 56 | + | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| |||
0 commit comments