This is an automated email from the ASF dual-hosted git repository.

willbarrett pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 52d7671  Remove incorrect comment (#10652)
52d7671 is described below

commit 52d7671204a1c55c9c4262319f0d5101f0c775e4
Author: Will Barrett <[email protected]>
AuthorDate: Mon Aug 24 10:40:36 2020 -0700

    Remove incorrect comment (#10652)
---
 superset/charts/api.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/superset/charts/api.py b/superset/charts/api.py
index d913801..4fd70f4 100644
--- a/superset/charts/api.py
+++ b/superset/charts/api.py
@@ -565,7 +565,7 @@ class ChartRestApi(BaseSupersetModelRestApi):
             }
             cache_chart_thumbnail.delay(**kwargs)
             return self.response(
-                202, cache_key=cache_key, chart_url=chart_url, 
image_url=image_url,
+                202, cache_key=cache_key, chart_url=chart_url, 
image_url=image_url
             )
 
         return trigger_celery()
@@ -614,8 +614,6 @@ class ChartRestApi(BaseSupersetModelRestApi):
         if not chart:
             return self.response_404()
 
-        # TODO make sure the user has access to the chart
-
         # fetch the chart screenshot using the current user and cache if set
         img = ChartScreenshot.get_from_cache_key(thumbnail_cache, digest)
         if img:

Reply via email to