get_signatures_for_address is missing min_context_slot param

[don1989]

Hello

The min_context_slot param is missing in the get_signatures_for_address method. I noticed that it is present in the config object that we send in the request:

class RpcSignaturesForAddressConfig:
    def __init__(
        self,
        before: Optional[Signature] = None,
        until: Optional[Signature] = None,
        limit: Optional[int] = None,
        commitment: Optional[CommitmentLevel] = None,
        min_context_slot: Optional[int] = None,
    ): ...

However, I don't understand why min_context_slot is missing in get_signatures_for_address and why we're not passing it to the config:

  def _get_signatures_for_address_body(
      self,
      address: Pubkey,
      before: Optional[Signature],
      until: Optional[Signature],
      limit: Optional[int],
      commitment: Optional[Commitment],
  ) -> GetSignaturesForAddress:
      commitment_to_use = _COMMITMENT_TO_SOLDERS[commitment or self._commitment]
      config = RpcSignaturesForAddressConfig(before=before, until=until, limit=limit, commitment=commitment_to_use)
      return GetSignaturesForAddress(address, config)

Is it possible to add it in the next release?

For reference: https://solana.com/docs/rpc/http/getsignaturesforaddress

Thanks

[don1989]

Its simple enough that I can do a PR. Can I be granted the relevant permissions @michaelhly please?

[michaelhly]

@don1989 Yes. Just add the min_context_slot as an optional argument to the function signature in the following places:

solana-py/src/solana/rpc/api.py

Line 312 in ff9af68

def get_signatures_for_address(

solana-py/src/solana/rpc/async_api.py

Line 324 in ff9af68

async def get_signatures_for_address(

solana-py/src/solana/rpc/core.py

Line 214 in ff9af68

def _get_signatures_for_address_body(

and then fix any relevant tests, lint and formatting issues.

Please fork this repo and contribute a pull request pointing to michaelhly/solana-py.