Should getWallets require a parameter of factoryAddress, similar to getWallet?

[chrisli30]

Since we added a factoryAddress param to getWallet, getWallets seems to need it too.

Currently,

getWallet(salt, factoryAddress, options);

getWallets(options);

[v9n]

@chrisli30 We already defined it but just haven't implement. In the avs.proto right now

message GetWalletReq {
  string salt = 1;
  // this is the factory address for the wallet, when leaving its empty, we will use our default factory address
  string factory_address = 2;
}

message ListWalletReq {
  // filter out by factory address or salt
  // otherwise return all the wallet
  string factory = 1; # TODO: VInh to rename this to factory_address for consistent
  string salt = 2;
}

We just haven't implement it

[v9n]

This is implemented. We can now filter by factory address https://github.com/AvaProtocol/EigenLayer-AVS/pull/84/files#diff-39a3b937dbf5b5c5180a7b4463059d0d2f310ef3a132fc8833dd1d63ffd00bc3R244-R247