Ignite Tutorial on BankKeeper Use is Flawed

[slendermaan]

Describe the bug

Credit to : HelloBloc

The Ignite tutorial on the use of BankKeeper is flawed, specifically, some of the BankKeeper.SendCoinxx operations are missing important error handling.
fix

if isFound {
         ...
        k.bankKeeper.SendCoins(ctx, buyer, owner, bid)
    }

to

if isFound {
         ...
        err := k.bankKeeper.SendCoins(ctx, buyer, owner, bid)
		if err != nil {
			return nil, err
		}
    }

To reproduce
Steps to reproduce the behavior:

1. Build NameService
2. Initialize a test account without token in config.yml
3. call BuyName, you can find that it can be called normally even without money.

Doc Url
https://docs.ignite.com/guide/nameservice/keeper

Associated Projects
Currently the following projects may have the same error due to tutorial issues, some of them have been fixed
sourcegraph

[tbruyelle]

Hey @slendermaan thank you very much for your report. I just pushed a PR to fix that.

[slendermaan]

👍