Skip to content

🐛 get the correct plotly_domain, not the "session"'s#363

Merged
chriddyp merged 3 commits into
masterfrom
fix-offline-enterprise-url
Dec 1, 2015
Merged

🐛 get the correct plotly_domain, not the "session"'s#363
chriddyp merged 3 commits into
masterfrom
fix-offline-enterprise-url

Conversation

@chriddyp
Copy link
Copy Markdown
Member

@chriddyp chriddyp commented Dec 1, 2015

No description provided.

@chriddyp
Copy link
Copy Markdown
Member Author

chriddyp commented Dec 1, 2015

cc @theengineear , @etpinard , @cldougl

Comment thread plotly/offline/offline.py
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mind tagging this with an issue in this repo? 🐹

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@theengineear
Copy link
Copy Markdown
Contributor

Loving the commit emojis :)

Non-blocking --> 💃

chriddyp added a commit that referenced this pull request Dec 1, 2015
🐛 get the correct plotly_domain, not the "session"'s
@chriddyp chriddyp merged commit 20102d6 into master Dec 1, 2015
@chriddyp chriddyp deleted the fix-offline-enterprise-url branch December 1, 2015 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants