Param handling (#348)

* replaced params with get_params which is standard

* lint

* refactors

* Bump black from 22.10.0 to 22.12.0 (#346)

Bumps [black](https://github.com/psf/black) from 22.10.0 to 22.12.0.
- [Release notes](https://github.com/psf/black/releases)
- [Changelog](https://github.com/psf/black/blob/main/CHANGES.md)
- [Commits](psf/black@22.10.0...22.12.0)

---
updated-dependencies:
- dependency-name: black
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump types-setuptools from 65.6.0.1 to 65.6.0.2 (#347)

* Edge headers (#343)

* removed formatting from docs and unwated comment dupe

* Add headers from options

* Added options parameter to rest api functions.

* handle headers from options

* edge-headers push

* Attempting to use with options

* created response options class

* handle headers from options

* merge headers

* parameter type cleanup and None value handling in request.

* attempting to revert conf.py changes

* added concat_method to baseclient and test for requestoptions

* refactored and introduced more optionals for stricter use

* added type hinting to builder method returns. removed optional from edge_headers method

* removed one example from ./example/launchpad and renamed function

* lint

* Update polygon/rest/base.py

remove redundancy.

* Update examples/launchpad/launchpad.py

Co-authored-by: jbonzo <[email protected]>

* param handling and refactor cleanup

* removed named param

* lint

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: jbonzo <[email protected]>

Author: 3 people

polygon/rest/base.py

@@ -74,11 +74,6 @@ def _get(
         raw: bool = False,
         options: Optional[RequestOptionBuilder] = None,
     ) -> Any:
-        if params is None:
-            params = {}
-        params = {str(k): str(v) for k, v in params.items() if v is not None}
-        logger.debug("_get %s params %s", path, params)
-
         option = options if options is not None else RequestOptionBuilder()
 
         resp = self.client.request(

polygon/rest/quotes.py

@@ -115,7 +115,7 @@ def get_real_time_currency_conversion(
         self,
         from_: str,
         to: str,
-        amount: float,
+        amount: Optional[float] = None,
         precision: Union[int, Precision] = 2,
         params: Optional[Dict[str, Any]] = None,
         raw: bool = False,
@@ -133,13 +133,10 @@ def get_real_time_currency_conversion(
         :return: Real-Time Currency Conversion
         """
         url = f"/v1/conversion/{from_}/{to}"
-        if params is None:
-            params = {}
-        params["amount"] = amount
-        params["precision"] = precision
+
         return self._get(
             path=url,
-            params=params,
+            params=self._get_params(self.get_real_time_currency_conversion, locals()),
             deserializer=RealTimeCurrencyConversion.from_dict,
             raw=raw,
             options=options,

polygon/rest/reference.py

@@ -152,7 +152,8 @@ def get_ticker_details(
 
     def get_ticker_events(
         self,
-        ticker: Optional[str] = None,
+        ticker: str,
+        types: Optional[str] = None,
         params: Optional[Dict[str, Any]] = None,
         raw: bool = False,
         options: Optional[RequestOptionBuilder] = None,
@@ -169,7 +170,7 @@ def get_ticker_events(
 
         return self._get(
             path=url,
-            params=params,
+            params=self._get_params(self.get_ticker_events, locals()),
             deserializer=TickerChangeResults.from_dict,
             result_key="results",
             raw=raw,
@@ -238,7 +239,7 @@ def get_ticker_types(
 
         return self._get(
             path=url,
-            params=params,
+            params=self._get_params(self.get_ticker_types, locals()),
             deserializer=TickerTypes.from_dict,
             raw=raw,
             result_key="results",
@@ -451,7 +452,7 @@ def get_exchanges(
 
         return self._get(
             path=url,
-            params=params,
+            params=self._get_params(self.get_exchanges, locals()),
             deserializer=Exchange.from_dict,
             raw=raw,
             result_key="results",

…ks/vX/reference/tickers/META/events.json …ers/META/events&types=ticker_change.jsontest_rest/mocks/vX/reference/tickers/META/events.json renamed to test_rest/mocks/vX/reference/tickers/META/events&types=ticker_change.json test_rest/mocks/vX/reference/tickers/META/events.json renamed to test_rest/mocks/vX/reference/tickers/META/events&types=ticker_change.json

@@ -239,7 +239,7 @@ def test_get_ticker_types(self):
         self.assertEqual(types, expected)
 
     def test_get_ticker_events_ticker_change(self):
-        events = self.c.get_ticker_events(ticker="META")
+        events = self.c.get_ticker_events(ticker="META", types="ticker_change")
         expected = TickerChangeResults(
             name="Meta Platforms, Inc. Class A Common Stock",
             figi="BBG000MM2P62",