Skip to content
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.

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

Reorganization [part 1] #1

Merged
merged 32 commits into from
Nov 13, 2015
Merged

Reorganization [part 1] #1

merged 32 commits into from
Nov 13, 2015

Conversation

etpinard
Copy link
Contributor

@bpostlethwaite @alexcjohnson

The most interesting part of this PR is under tasks/, please see comments below.

- run with `npm run watch`
- supports 'require' (for test_dashboard)
@@ -1 +1,6 @@
node_modules

build/*
!build/README.md
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'd vote for ignoring dist/* as well and have us git add -f dist/ when we bump the version.

Thoughts?

Copy link
Member

Choose a reason for hiding this comment

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

yes lets do this - we can always turn it off if we don't like it

@bpostlethwaite
Copy link
Member

💃

etpinard added a commit that referenced this pull request Nov 13, 2015
Reorganization [part 1]
@etpinard etpinard merged commit ad6f080 into master Nov 13, 2015
@etpinard etpinard deleted the reorg branch November 13, 2015 23:48
etpinard pushed a commit that referenced this pull request Aug 2, 2016
Legend item wrap with layout.legend.orientation = h
etpinard pushed a commit that referenced this pull request Feb 8, 2017
@apalchys apalchys mentioned this pull request Aug 22, 2017
monfera added a commit that referenced this pull request Oct 5, 2017
alexcjohnson pushed a commit that referenced this pull request May 30, 2018
* treat zero as valid fixes #2660

* PR suggestions: toBe instead of toEqual
alexcjohnson added a commit that referenced this pull request May 30, 2018
Fix for number 0 treated as invalid in hover (#1)
etpinard pushed a commit that referenced this pull request Oct 5, 2018
etpinard pushed a commit that referenced this pull request Feb 12, 2019
* Fix bug in scattergl plot
* Added tests for scattergl plot update
archmoj pushed a commit that referenced this pull request May 26, 2020
Pull all changes from original plotly to the fork
archmoj pushed a commit that referenced this pull request Sep 24, 2020
Fixed the per-element opacity on Surface plots
archmoj pushed a commit that referenced this pull request Sep 29, 2020
archmoj pushed a commit that referenced this pull request Oct 7, 2021
ayjayt pushed a commit to geopozo/plotly.js that referenced this pull request Jan 25, 2024
Update to your pull-request for making es5/require work.
gvwilson pushed a commit that referenced this pull request Nov 18, 2024
@marthacryan marthacryan mentioned this pull request Jan 17, 2025
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.

4 participants