pankajkoti commented on PR #31482: URL: https://github.com/apache/airflow/pull/31482#issuecomment-1561849820
> > IMO this is not a requirement to merge this fix, but it's a nice enhancement for the AWS operators links which can be implemented separately. > > @ferruzzi @o-nikolas wdyt? > > I'm not thrilled with the idea of knowingly vending a broken link. What about a compromise for now where if the link isn't available, return a string along the lines of "N/A"?I Love the idea of disabling the link if it's not available, but a static string message would be better than a dead link IMHO +1 to this idea and perhaps some more description in the string to include something like "that you have disabled logging and hence ...." based upon the comment https://github.com/apache/airflow/issues/31480#issuecomment-1560723485 might hint the user why it is N/A, no? -- 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]
