BlockHeader vs BlockPayload

[mversic]

Let's revise which fields go into which of these 2 structures. This is the current state:

    pub struct BlockHeader {
        pub height: u64,
        pub timestamp_ms: u64,
        pub previous_block_hash: Option<HashOf<VersionedSignedBlock>>,
        pub transactions_hash: Option<HashOf<MerkleTree<VersionedSignedTransaction>>>,
        pub commit_topology: Vec<peer::PeerId>,
        pub view_change_index: u64,
        pub consensus_estimation_ms: u64,
    }

    pub struct BlockPayload {
        pub header: BlockHeader,
        pub transactions: Vec<TransactionValue>,
        pub event_recommendations: Vec<Event>,
    }

[mversic]

IMO commit_topology should go into BlockPayload, not BlockHeader