Skip to content

Cleanup for the StateDecodeParams implementation #5937

@akaladarshi

Description

@akaladarshi

Summary

Once the StateDecodeParams API support for all actor mentioned in the #5680 is implemented, we need to cleanup few things:

  • Decide a files names for the implementation:

    • actor-name_actor
    • actor-name_methods
    • actor-name_params
    • actor-name_actor_params
    • actor-name_actor_method_params
  • Use enum for version instead of u64 to track all the implementation

    Originally posted by @LesnyRumcajs in feat: add support for verified reg actor in state decode params #5896 (comment)

  • Refactor the method api_compare_tests state_decode_params tests functions into separate file

    Originally posted by @LesnyRumcajs in feat: add support for verified reg actor in state decode params #5896 (comment)

  • Use fil_actor_<actor_name>_state for method number instead of magic numbers

  • Update the ChangeLog file for stateDecodeParams API

  • Combine remaining individual files of same in lotus_json/actor i.e. account_authenticate_params into a single file

  • Add Exec4 in the v10 of the fil-actor-state and update the init actor

  • Add support for the v8 to v10 for the fil_actor_account_state

    Note: Earlier it was not added because the files/types were not exposed

Metadata

Metadata

Assignees

Labels

Type: TaskDiscrete task to implement

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions