Naming conflict two transaction versions with different values.

[darkfriend77]

transaction_version

substrate/bin/node/runtime/src/lib.rs

Line 115 in c0cb704

transaction_version: 1,

const TRANSACTION_VERSION: u8 = 4;

substrate/primitives/runtime/src/generic/unchecked_extrinsic.rs

Line 33 in c0cb704

const TRANSACTION_VERSION: u8 = 4;

Why do they have the same name? One is used for the extrinsic start and the other one is used in the payload, so both are represented in the submitextrinsic byte string.

I would suggest to either rename the one in the unchecked_extrinsic or to unify them.