Use web_url in response instead of plotly_domain in session\nplotly_domain is not necessarily the same as plotly_api_domain, so parsing grid_id into plotly_domain could give the wrong URL

chriddyp · chriddyp · commit 096073eb667f · 2014-12-10T20:37:12.000-08:00
diff --git a/plotly/plotly/plotly.py b/plotly/plotly/plotly.py
@@ -840,15 +840,13 @@ def upload(cls, grid, filename,
 
         response_columns = res['file']['cols']
         grid_id = res['file']['fid']
+        grid_url = res['file']['web_url']
 
         # mutate the grid columns with the id's returned from the server
         cls._fill_in_response_column_ids(grid, response_columns, grid_id)
 
         grid.id = grid_id
 
-        plotly_domain = get_config()['plotly_domain']
-        grid_url = '{0}/~{1}'.format(plotly_domain, grid_id.replace(':', '/'))
-
         if meta is not None:
             meta_ops.upload(meta, grid=grid)
 
