remove deprecated code

Author: rmorshea

docs/source/about/changelog.rst

@@ -28,6 +28,14 @@ Unreleased
 - :pull:`876` - ``idom.widgets.hotswap``. The function has no clear uses outside of some
   internal applications. For this reason it has been deprecated.
 
+**Removed**
+
+- :pull:`886` - Ability to access element value from events via `event['value']` key.
+  Instead element value should be accessed via `event['target']['value']`. Originally
+  deprecated in :ref:`v0.34.0`.
+- :pull:`886` - old misspelled option ``idom.config.IDOM_WED_MODULES_DIR``. Originally
+  deprecated in :ref:`v0.36.1`.
+
 
 v0.43.0
 -------

src/idom/_option.py

@@ -120,19 +120,14 @@ def __repr__(self) -> str:
 
 
 class DeprecatedOption(Option[_O]):  # pragma: no cover
-    def __init__(self, new_opt: Option[_O], name: str) -> None:
-        # copy over attrs
-        attrs = new_opt.__dict__.copy()
-        attrs.pop("_current", None)
-        self.__dict__.update(new_opt.__dict__)
-        # then set the ones needed here
-        self._name = name
-        self._new_opt = new_opt
+    def __init__(self, message: str, *args: Any, **kwargs: Any) -> None:
+        self._deprecation_message = message
+        super().__init__(*args, **kwargs)
 
     @property  # type: ignore
     def _current(self) -> _O:
         warn(
-            f"{self.name!r} has been renamed to {self._new_opt.name!r}",
+            self._deprecation_message,
             DeprecationWarning,
             stacklevel=_frame_depth_in_module() + 1,
         )

src/idom/config.py

@@ -6,7 +6,6 @@
 from pathlib import Path
 from tempfile import TemporaryDirectory
 
-from ._option import DeprecatedOption as _DeprecatedOption
 from ._option import Option as _Option
 
 
@@ -54,12 +53,6 @@
 set of publically available APIs for working with the client.
 """
 
-IDOM_WED_MODULES_DIR: _Option[Path] = _DeprecatedOption(
-    new_opt=IDOM_WEB_MODULES_DIR,
-    name="IDOM_WED_MODULES_DIR",
-)
-"""This has been renamed to :data:`IDOM_WEB_MODULES_DIR`"""
-
 IDOM_TESTING_DEFAULT_TIMEOUT = _Option(
     "IDOM_TESTING_DEFAULT_TIMEOUT",
     5.0,

src/idom/core/_event_proxy.py

@@ -26,7 +26,6 @@
 from idom.config import IDOM_CHECK_VDOM_SPEC, IDOM_DEBUG_MODE
 from idom.utils import Ref
 
-from ._event_proxy import _wrap_in_warning_event_proxies
 from .hooks import LifeCycleHook
 from .types import (
     ComponentType,
@@ -99,7 +98,7 @@ async def deliver(self, event: LayoutEventMessage) -> None:
 
         if handler is not None:
             try:
-                await handler.function(_wrap_in_warning_event_proxies(event["data"]))
+                await handler.function(event["data"])
             except Exception:
                 logger.exception(f"Failed to execute event handler {handler}")
         else:

src/idom/core/layout.py

@@ -3,7 +3,7 @@
 
 import pytest
 
-from idom._option import Option
+from idom._option import DeprecatedOption, Option
 
 
 def test_option_repr():
@@ -99,3 +99,10 @@ def test_option_subscribe():
 
     opt.unset()
     assert calls == ["default", "new-1", "new-2", "default"]
+
+
+def test_deprecated_option():
+    opt = DeprecatedOption("is deprecated!", "A_FAKE_OPTION", None)
+
+    with pytest.warns(DeprecationWarning, match="is deprecated!"):
+        opt.current