remove deprecated fields in ledger command, fix for #3214

[ckeshava]

trafficstars

I removed the following fields from the ledger command - accepted, seq, hash and totalCoins.

Some of these variable names (totalCoins, seq) are used in managing the database but I don't think there is any redundancy in the creation/updation/deletion of the tables.

[nbougalis]

I don't know if we can just remove the deprecated field; someone might still be relying on it. Do we know how long it's been marked DEPRECATED for? @mDuo13, any objections?

[mDuo13]

Per #3214, the fields have been marked as deprecated dating back to at least 2015. I think we can just remove them.

[intelliot]

@mDuo13 any thoughts on jss::hash?

[mDuo13]

The hash field in ledger_request should stay (for now). That command has its own problems, but I don't think it uses the shared ledger deserialization that this ticket is about.

[intelliot]

This is slated for inclusion in 1.12 (not 1.11) unless anyone comments here to request otherwise.

[intelliot]

Released in 1.12.0.