ryanahamilton opened a new pull request #14601:
URL: https://github.com/apache/airflow/pull/14601


   After fixing and refactoring the "demo mode" feature in #14595, I feel that 
it would be more productive to replace that PR with this one that removes the 
feature instead. 
   
   - It has very little documentation
   - It has been largely broken for an unknown period of time without any issue 
reports. I only encountered it because I wanted to refactor its implementation.
   - [As illustrated by 
@ashb](https://github.com/apache/airflow/pull/14595#issuecomment-790581682) it 
doesn't do a really great job if someone were to capture a screenshot from 
remote demonstration.
   - As pointed out by @HaloKo4, only being able to activate it from the config 
doesn't make it a very convenient feature that can easily be enabled/disabled.
   
   If this is a feature people would like to see, I think it would be better to 
see a new implementation proposed that addresses these shortcomings.


----------------------------------------------------------------
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.

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


Reply via email to