Move struct parsers should to rpcclient/ packages #93

Open
opened 2024-05-07 10:04:28 +00:00 by fyrchik · 0 comments
Owner

Consider https://git.frostfs.info/TrueCloudLab/frostfs-contract/src/branch/master/frostfsid/client/client.go#L249. parseSubjectExtended is used privately and should be duplicated in the client code (see TrueCloudLab/frostfs-node#1117).

A better approach would be to have sth like ParseSubjectExtended([]stackitem.Item) or UnwrapSubjectExtended(*result.Invoke, error) together with the autogenerated code.
In the bright future, it will surely support generating custom structs.

Consider https://git.frostfs.info/TrueCloudLab/frostfs-contract/src/branch/master/frostfsid/client/client.go#L249. `parseSubjectExtended` is used privately and should be duplicated in the client code (see https://git.frostfs.info/TrueCloudLab/frostfs-node/pulls/1117). A better approach would be to have sth like `ParseSubjectExtended([]stackitem.Item)` or `UnwrapSubjectExtended(*result.Invoke, error)` together with the autogenerated code. In the bright future, it will surely support generating custom structs.
fyrchik changed title from Struct parsers should belong to rpcclient/ packages to Move struct parsers should to rpcclient/ packages 2024-05-07 10:04:40 +00:00
fyrchik added the
refactoring
internal
labels 2024-05-07 11:43:27 +00:00
fyrchik added this to the v0.20.0 milestone 2024-08-19 08:55:18 +00:00
fyrchik modified the milestone from v0.20.0 to v0.21.0 2024-09-30 09:39:09 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-contract#93
No description provided.