amoghrajesh commented on code in PR #70298:
URL: https://github.com/apache/airflow/pull/70298#discussion_r3774741772


##########
dev/registry/extract_parameters.py:
##########
@@ -432,6 +433,11 @@ def is_durable_capable(cls: type, resumable_mixin: type | 
None) -> bool:
     return "execute_resumable" in source
 
 
+def supports_deferrable(cls: type) -> bool:
+    """Return True if a class exposes a `deferrable` constructor parameter."""

Review Comment:
   Handled in 03b18d4da9



##########
registry/src/js/provider-detail.js:
##########
@@ -25,6 +25,9 @@
   var extraDepsList = document.getElementById('extra-deps-list');
   var extrasDataEl = document.getElementById('extras-data');
   var moduleSearch = document.getElementById('module-search');
+  var durableOnlyFilter = document.getElementById('durable-only-filter');
+  var deferrableOnlyFilter = document.getElementById('deferrable-only-filter');
+  var moduleTabs = document.querySelectorAll('.module-tab');

Review Comment:
   Handled in [handling simple review 
comments](https://github.com/apache/airflow/pull/70298/commits/7d3cba4b39510a6badba4d44c7314bc10a05c4dd)



##########
registry/src/css/main.css:
##########
@@ -3468,8 +3490,18 @@ main {
   font-size: var(--text-xs);
   font-weight: var(--font-medium);
   border-radius: var(--radius-lg);
+}
+
+.provider-detail-page .modules .module .content h3 .durable-badge {
   background: color-mix(in srgb, var(--color-teal-500) 15%, transparent);
-  color: var(--color-teal-400);
+  color: var(--color-teal-600);
+  color: light-dark(var(--color-teal-600), var(--color-teal-400));

Review Comment:
   Handled in [handling simple review 
comments](https://github.com/apache/airflow/pull/70298/commits/7d3cba4b39510a6badba4d44c7314bc10a05c4dd)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to