Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor(auth): use collections for Account state management #16016

Merged
merged 34 commits into from
May 26, 2023
Merged

refactor(auth): use collections for Account state management #16016

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
130df6d
unembed keeper from msg server, create query server to implement the …
Apr 28, 2023
6c8ea95
change collection name to just "Params"
Apr 28, 2023
fc2bbe4
Merge branch 'main' into tip/coll/auth_accounts_2
testinginprod Apr 28, 2023
0b547dd
partially move accounts to use collections
Apr 28, 2023
8686088
fix sigverify test
Apr 28, 2023
e3ed273
fix sigverify test2
Apr 28, 2023
1f33e06
fix vesting tests
Apr 30, 2023
52de1f7
Merge branch 'main' into tip/coll/auth_accounts_3
May 2, 2023
a82c815
fix more tests
May 3, 2023
b351257
remove more unneeded things
May 3, 2023
2fe4672
more code to use collections
May 3, 2023
8704230
remove more unneeded things
May 3, 2023
fc4ad40
adjust gas tests
May 3, 2023
074c06d
Merge branch 'main' into tip/coll/auth_accounts_3
May 6, 2023
bf23c61
remove not relevant tests
May 6, 2023
c350695
Merge branch 'main' into tip/coll/auth_accounts_3
May 11, 2023
b8ed5e1
remove store decoder
May 11, 2023
6e11ac5
remove APIs that are not used anymore
May 11, 2023
49004af
chore lint
May 11, 2023
4d19a0c
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 11, 2023
835c2e9
chore: lint
May 11, 2023
4225d10
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 15, 2023
a1796e5
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 15, 2023
0a9ef82
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 19, 2023
cde0471
fix merge issues
May 19, 2023
82f566f
hack to make the test pass
May 19, 2023
e93e41f
chore: CHANGELOG.md
May 19, 2023
de0ea3a
remove: SetParams
May 19, 2023
6cc4cdb
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 19, 2023
d66986f
Merge branch 'main' into tip/coll/auth_accounts_3
May 22, 2023
3333da1
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 24, 2023
a18db18
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 25, 2023
b552c96
Merge branch 'main' into tip/coll/auth_accounts_3
testinginprod May 26, 2023
ddfaf08
fix merge
May 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into tip/coll/auth_accounts_3 
# Conflicts:
#	x/auth/keeper/grpc_query.go
#	x/auth/keeper/keeper.go
  • Loading branch information
unknown unknown committed May 22, 2023
commit d66986f6f5bad2dde9d883dfc454d7c4289d707e
4 changes: 2 additions & 2 deletions x/auth/keeper/grpc_query.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ func (s queryServer) Account(ctx context.Context, req *types.QueryAccountRequest
return nil, status.Error(codes.InvalidArgument, "Address cannot be empty")
}

addr, err := s.k.StringToBytes(req.Address)
addr, err := s.k.addressCodec.StringToBytes(req.Address)
if err != nil {
return nil, err
}
Expand Down Expand Up @@ -214,7 +214,7 @@ func (s queryServer) AccountInfo(ctx context.Context, req *types.QueryAccountInf
return nil, status.Error(codes.InvalidArgument, "address cannot be empty")
}

addr, err := s.k.StringToBytes(req.Address)
addr, err := s.k.addressCodec.StringToBytes(req.Address)
if err != nil {
return nil, err
}
Expand Down
2 changes: 1 addition & 1 deletion x/auth/keeper/keeper.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ func NewAccountKeeper(
sb := collections.NewSchemaBuilder(storeService)

ak := AccountKeeper{
Codec: authcodec.NewBech32Codec(bech32Prefix),
addressCodec: authcodec.NewBech32Codec(bech32Prefix),
bech32Prefix: bech32Prefix,
storeService: storeService,
proto: proto,
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.