aayushbisen commented on issue #50737:
URL: https://github.com/apache/airflow/issues/50737#issuecomment-3174228186

   In [airflow/providers/discord/notifications/discord.py], we're currently 
using a raw GitHub URL for the `ICON_URL`:
   
   ```python
   ICON_URL: str = (
       
"https://raw.githubusercontent.com/apache/airflow/main/airflow-core/src/airflow/ui/public/pin_100.png";
   )
   ```
   
   I feel the current implementation is correct and doesn't need to be changed 
to use the GitHub API. The GitHub API would only add unnecessary complexity and 
potential rate limiting issues for this use case.
   
   If there's a specific reason to use the GitHub API that I'm missing, please 
let me know!
   
   


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to