Thanks to visit codestin.com
Credit goes to github.com

Skip to content

2.0.0 #658

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Jan 19, 2017
Merged

2.0.0 #658

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
e68cbf5
Try getting the error message in response_handler
alexandresobolevski Dec 21, 2016
19162af
Did not mean to commit this
alexandresobolevski Dec 21, 2016
35f3bcd
Always raise PlotlyRequestError
alexandresobolevski Dec 22, 2016
cf4d831
Change expected exception
alexandresobolevski Dec 22, 2016
cf159d6
Factor out `get_config` and `get_credentials`.
theengineear Dec 22, 2016
15d3c8b
Generalize our `PlotlyRequestError`.
theengineear Dec 27, 2016
c0c6e6c
Create organized api modules :tada:!
theengineear Dec 27, 2016
403034e
Use the new api definitions in all api calls.
theengineear Dec 27, 2016
a320248
Use `api.v2.plot_schema` to get GRAPH_REFERENCE.
theengineear Dec 27, 2016
4304f0e
:wrench: Allow `proxy_*` in `sign_in`.
theengineear Dec 27, 2016
6634971
Use requests.compat.json instead of plain json.
theengineear Dec 29, 2016
9832c11
:dolls: Remove outdated comment.
theengineear Jan 4, 2017
70e7211
Move FF from `tools.py` to `/figure_factory` pkg.
theengineear Jan 6, 2017
fb02d40
Stop doing weird protected imports for FF stuff.
theengineear Jan 6, 2017
c859ac0
Fix FF docs.
theengineear Jan 6, 2017
867fc11
Some pep:8ball: changes.
theengineear Jan 6, 2017
a617845
Add an `optional_imports` module.
theengineear Oct 2, 2015
51db9bc
Use `optional_imports` instead of `_*_imported`.
theengineear Jan 9, 2017
b7af7e3
Update CHANGELOG.md.
theengineear Jan 11, 2017
65ee9d2
Ensure `plot` sends layout along.
theengineear Jan 17, 2017
8fd6c8f
Namespace `session` import in `plotly.py`.
theengineear Jan 18, 2017
8ea90fe
Add `users.current` endpoint to v2 api.
theengineear Jan 18, 2017
0b6e010
Raise `PlotlyError` if sign_in fails.
theengineear Jan 18, 2017
1a6605f
Fail if user tries to import FF without `numpy`.
theengineear Jan 18, 2017
0259649
Move all FF tests into `test_optional`.
theengineear Jan 18, 2017
f1b4981
Version bump —> 2.0.0 (see CHANGELOG.md).
theengineear Jan 18, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Always raise PlotlyRequestError
  • Loading branch information
@alexandresobolevski @theengineear
alexandresobolevski authored and theengineear committed Jan 18, 2017
commit 35f3bcdc74168a47dddc86da6fd16d4342b7ed4c
8 changes: 5 additions & 3 deletions plotly/plotly/plotly.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1318,9 +1318,11 @@ def response_handler(cls, response):
errors = parsed_response.get('errors')
message = errors[-1]['message']
except:
raise requests_exception

raise exceptions.PlotlyError(message)
# otherwise raise a generic error
raise exceptions.PlotlyRequestError(requests_exception)
else:
requests_exception.message = message
raise exceptions.PlotlyRequestError(requests_exception)

if ('content-type' in response.headers and
'json' in response.headers['content-type'] and
Expand Down