desmos-labs / desmos

Improving the well-being of users on social networks through the blockchain technology.
https://desmos.network
Apache License 2.0
103 stars 45 forks source link

fix: fix missing account number of account creation #1250

Closed dadamu closed 9 months ago

dadamu commented 9 months ago

Description

Closes: #XXXX This PR fixes the issue that account has no account number since using authtypes.NewBaseAccountWithAddress. To solve it, we should use AccountKeeper.NewAccountWithAddress instead, to make sure account number is set.


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

codecov[bot] commented 9 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Files Coverage Δ
x/profiles/keeper/msg_server_dtag_transfers.go 77.27% <100.00%> (ø)
x/profiles/keeper/msgs_server_profile.go 80.26% <100.00%> (-3.08%) :arrow_down:

:loudspeaker: Thoughts on this report? Let us know!.