feat: update x/bankplus for deprecation

[jaeseung-bae]

Description

closes: #1212

• NOTE: This PR has proto breaking change. lbm/bankplus/v1/bankplus.proto has been deleted for deprecation.

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

• I followed the contributing guidelines and code of conduct.
• I have added a relevant changelog to CHANGELOG.md
• I have added tests to cover my changes.
• I have updated the documentation accordingly.
• I have updated API documentation client/docs/swagger-ui/swagger.yaml

[0Tech]

We should consider implementing the migration on the app. You may try to implement it on simapp, and if it works, you can open the corresponding PR on Finschia/finschia.

[ulbqb]

How about leaving deprecator in bankplus because it's used by finschia/finschia? And proto files shouldn't be removed.

[jaeseung-bae]

How about leaving deprecator in bankplus because it's used by finschia/finschia? And proto files shouldn't be removed.

I'm thinking of closing this PR when another PR on finschia/finschia is ready.
As @0Tech recommended, I'll make another PR just for bankplus deprecation logic on finschia/finschia.
finschia-sdk will look like it's never had bankplus before.

[ulbqb]

In my opinion, the repo that has better to have deprecator is finschia-sdk not finschia. If finschia, that makes it hard to follow history of bankplus and that makes proto files scattered.

[jaeseung-bae]

tter to have deprecator is finschia-sdk not finschia. If finschia, that makes it hard to follow history of bankplus and that makes proto files scattered.

I agree. It seems better to keep bankplus logic here for history.

[ulbqb]

Is CI testing bankplus? I suppose setting .github/workflow/test.yml is needed.

[ulbqb]

LGTM