refactor: use single Oracle class for Price Oracles (#704)

* use single Oracle class for get_aggregate_price and ledger_entry to resolve duplicate name conflict.

tests/unit/models/requests/test_get_aggregate_price.py

@@ -2,7 +2,7 @@
 
 from xrpl.models import XRPLModelException
 from xrpl.models.requests import GetAggregatePrice
-from xrpl.models.requests.get_aggregate_price import Oracle
+from xrpl.models.requests.ledger_entry import Oracle
 
 _ACCT_STR_1 = "rBwHKFS534tfG3mATXSycCnX8PAd3XJswj"
 _ORACLE_DOC_ID_1 = 1

xrpl/models/requests/get_aggregate_price.py

@@ -8,8 +8,7 @@
 from dataclasses import dataclass, field
 from typing import Dict, List, Optional
 
-from typing_extensions import TypedDict
-
+from xrpl.models.requests.ledger_entry import Oracle
 from xrpl.models.requests.request import Request, RequestMethod
 from xrpl.models.required import REQUIRED
 from xrpl.models.utils import require_kwargs_on_init
@@ -49,15 +48,3 @@ def _get_errors(self: GetAggregatePrice) -> Dict[str, str]:
                 "GetAggregatePrice"
             ] = "Oracles array must contain at least one element"
         return errors
-
-
-@require_kwargs_on_init
-@dataclass(frozen=True)
-class Oracle(TypedDict):
-    """Represents one Oracle element. It is used in GetAggregatePrice request"""
-
-    oracle_document_id: int = REQUIRED  # type: ignore
-    """A unique identifier of the price oracle for the Account"""
-
-    account: str = REQUIRED  # type: ignore
-    """The XRPL account that controls the Oracle object"""