passionworkeer commented on issue #51102: URL: https://github.com/apache/airflow/issues/51102#issuecomment-4050253401
I’d keep the first pass narrow: add an optional storage_path (or provider config-derived default) in AppriseHook, build AppriseAsset(storage_path=..., storage_mode=AUTO) when it is present, and preserve current behavior when it is absent. That keeps this backward-compatible while enabling Matrix login caching. A good regression test would instantiate the hook twice with the same temp storage path and assert the asset is created with persistent storage enabled, without changing notification semantics for other providers. -- 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]
