Add docs to get_account_transactions and fix Sphinx dependency issues

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Function to parse the final account balances from a transaction's metadata
 - Function to parse order book changes from a transaction's metadata
 - Support for Ed25519 seeds that don't use the `sEd` prefix
+- Add the optional `marker` field to `get_account_transactions` to paginate through results [#462]
 
 ### Fixed:
 - Typing for factory classmethods on models

xrpl/account/transaction_history.py

@@ -1,6 +1,6 @@
 """High-level methods to obtain information about account transaction history."""
 import asyncio
-from typing import Any, Dict, List
+from typing import Any, Dict, List, Optional
 
 from xrpl.asyncio.account import transaction_history
 from xrpl.clients.sync_client import SyncClient
@@ -24,36 +24,46 @@ def get_latest_transaction(account: str, client: SyncClient) -> Response:
     return asyncio.run(transaction_history.get_latest_transaction(account, client))
 
 
-def get_account_transactions(address: str, client: SyncClient) -> List[Dict[str, Any]]:
+def get_account_transactions(
+    address: str, client: SyncClient, marker: Optional[Any] = None
+) -> List[Dict[str, Any]]:
     """
     Query the ledger for a list of transactions that involved a given account.
 
     Args:
         address: the account to query.
         client: the network client used to make network calls.
+        marker: fetches the next set of data from the server. The type of
+            marker is intentionally undefined in the spec and is chosen
+            by each server.
 
     Returns:
         The transaction history for the address.
 
     Raises:
         XRPLRequestFailureException: if the transaction fails.
     """
-    return asyncio.run(transaction_history.get_account_transactions(address, client))
+    return asyncio.run(
+        transaction_history.get_account_transactions(address, client, marker)
+    )
 
 
 def get_account_payment_transactions(
-    address: str, client: SyncClient
+    address: str, client: SyncClient, marker: Optional[Any] = None
 ) -> List[Dict[str, Any]]:
     """
     Query the ledger for a list of payment transactions that involved a given account.
 
     Args:
         address: the account to query.
         client: the network client used to make network calls.
+        marker: fetches the next set of data from the server. The type of
+            marker is intentionally undefined in the spec and is chosen
+            by each server.
 
     Returns:
         The payment transaction history for the address.
     """
     return asyncio.run(
-        transaction_history.get_account_payment_transactions(address, client)
+        transaction_history.get_account_payment_transactions(address, client, marker)
     )

xrpl/asyncio/account/transaction_history.py

@@ -1,5 +1,5 @@
 """High-level methods to obtain information about account transaction history."""
-from typing import Any, Dict, List, cast
+from typing import Any, Dict, List, Optional, cast
 
 from xrpl.asyncio.clients import Client, XRPLRequestFailureException
 from xrpl.core.addresscodec import is_valid_xaddress, xaddress_to_classic_address
@@ -33,14 +33,17 @@ async def get_latest_transaction(account: str, client: Client) -> Response:
 
 
 async def get_account_transactions(
-    address: str, client: Client
+    address: str, client: Client, marker: Optional[Any] = None
 ) -> List[Dict[str, Any]]:
     """
     Query the ledger for a list of transactions that involved a given account.
 
     Args:
         address: the account to query.
         client: the network client used to make network calls.
+        marker: fetches the next set of data from the server. The type of
+            marker is intentionally undefined in the spec and is chosen
+            by each server.
 
     Returns:
         The transaction history for the address.
@@ -50,25 +53,28 @@ async def get_account_transactions(
     """
     if is_valid_xaddress(address):
         address, _, _ = xaddress_to_classic_address(address)
-    request = AccountTx(account=address)
+    request = AccountTx(account=address, marker=marker)
     response = await client.request_impl(request)
     if not response.is_successful():
         raise XRPLRequestFailureException(response.result)
     return cast(List[Dict[str, Any]], response.result["transactions"])
 
 
 async def get_account_payment_transactions(
-    address: str, client: Client
+    address: str, client: Client, marker: Optional[Any] = None
 ) -> List[Dict[str, Any]]:
     """
     Query the ledger for a list of payment transactions that involved a given account.
 
     Args:
         address: the account to query.
         client: the network client used to make network calls.
+        marker: fetches the next set of data from the server. The type of
+            marker is intentionally undefined in the spec and is chosen
+            by each server.
 
     Returns:
         The payment transaction history for the address.
     """
-    all_transactions = await get_account_transactions(address, client)
+    all_transactions = await get_account_transactions(address, client, marker)
     return [tx for tx in all_transactions if tx["tx"]["TransactionType"] == "Payment"]