Closed nflaig closed 3 months ago
There should be a few routes we can remove from the code as those are no longer part of beacon api spec. There is no good reason to keep them around as they are unused and replaced by more forward compatible apis (v2).
E.g. getState (v1) has been removed from spec a while ago https://github.com/ChainSafe/lodestar/blob/8b59f500e22175d61155410db9fe587facf3e3c9/packages/api/src/beacon/routes/debug.ts#L120
getState
This should be done after https://github.com/ChainSafe/lodestar/pull/6749 is merged to avoid conflicts.
There should be a few routes we can remove from the code as those are no longer part of beacon api spec. There is no good reason to keep them around as they are unused and replaced by more forward compatible apis (v2).
E.g.
getState(v1) has been removed from spec a while ago https://github.com/ChainSafe/lodestar/blob/8b59f500e22175d61155410db9fe587facf3e3c9/packages/api/src/beacon/routes/debug.ts#L120This should be done after https://github.com/ChainSafe/lodestar/pull/6749 is merged to avoid conflicts.