Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

Multi msg type decoder and handler #204

Merged
merged 7 commits into from
Jul 14, 2023
Merged

Multi msg type decoder and handler #204

merged 7 commits into from
Jul 14, 2023

Conversation

hopeyen
Copy link
Collaborator

@hopeyen hopeyen commented Jul 11, 2023

Description

  • Update to the latest SDK changes, and then further to depend on open branch hope/subgraph-owner-check
  • Refactored message definitions, added VersionUpgradeMessage variant
    • Provided type definition, build, and verification (between sender, graph_account, subgraph id, and current hash)
    • maybe can be converted into a trait object with suitable trait bounds
  • Moved radio_msg_handler logic to be part of radio operator initialization to consist multi-msg types for graphcast agent's new companion waku_message_receiver
    • can be further refactored back to a method for radio operator
    • chains decoding and handling of message types for each waku message
  • Added one-shot binary, such that user can supply a set of fields in CLI to run the command to send a VersionUpgrade message
    • identifier: current subgraph deployment hash, used to locate the appropriate content topics, thus listening indexers
    • new_hash: new subgraph deployment hash, used to notify indexers the needed info to deploy
    • subgraph_id: confirms the relationship by supplying subgraph id
    • network: the indexing network of the subgraph
    • migrate_time: planned time to migrate usage over to the new deployment
    • graph_account: subgraph id's owner, used for verifications

Issue link (if applicable)

Resolves #203

Checklist

  • Are tests up-to-date with the new changes?
  • Are docs up-to-date with the new changes? (Open PR on docs repo if necessary)

@hopeyen hopeyen changed the title Hope/multi msg types Multi msg type decoder and handler Jul 11, 2023
@neriumrevolta neriumrevolta self-requested a review July 13, 2023 19:49
@hopeyen hopeyen merged commit a98fbff into dev Jul 14, 2023
6 checks passed
@hopeyen hopeyen deleted the hope/multi-msg-types branch July 14, 2023 12:24
@hopeyen hopeyen mentioned this pull request Jul 14, 2023
8 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add VersionUpgrade Message type
2 participants