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

[explore flow] handling duplicated column aliases #5778

Merged
merged 2 commits into from
Sep 19, 2018

Conversation

mistercrunch
Copy link
Member

closes #5739
screen shot 2018-08-29 at 10 50 48 pm

@elibrumbaugh
Copy link
Contributor

Hey @mistercrunch we should avoid using that red color for buttons unless clicking that button will lead to something critical such as deleting something.

@codecov-io
Copy link

Codecov Report

Merging #5778 into master will increase coverage by 0.01%.
The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5778      +/-   ##
==========================================
+ Coverage   63.77%   63.78%   +0.01%     
==========================================
  Files         364      364              
  Lines       23097    23112      +15     
  Branches     2572     2588      +16     
==========================================
+ Hits        14729    14743      +14     
- Misses       8353     8354       +1     
  Partials       15       15
Impacted Files Coverage Δ
superset/assets/src/visualizations/sunburst.js 0% <ø> (ø) ⬆️
...ets/src/SqlLab/components/ExploreResultsButton.jsx 82.79% <80%> (+0.37%) ⬆️
superset/assets/src/chart/chartReducer.js 16.94% <0%> (-0.6%) ⬇️
superset/assets/src/visualizations/country_map.js 0% <0%> (ø) ⬆️
superset/assets/src/visualizations/word_cloud.js
...t/assets/src/visualizations/wordcloud/WordCloud.js 0% <0%> (ø)
superset/viz.py 81.39% <0%> (ø) ⬆️
superset/models/annotations.py 93.1% <0%> (+0.24%) ⬆️
superset/views/annotations.py 76.08% <0%> (+0.53%) ⬆️
superset/assets/src/chart/Chart.jsx 58.4% <0%> (+0.7%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ada8b92...6317b04. Read the comment docs.

@mistercrunch
Copy link
Member Author

screen shot 2018-09-01 at 1 31 43 pm

@mistercrunch mistercrunch merged commit 75bc501 into apache:master Sep 19, 2018
@mistercrunch mistercrunch deleted the fix_5738 branch September 19, 2018 06:17
mistercrunch added a commit to lyft/incubator-superset that referenced this pull request Sep 21, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
mistercrunch added a commit to lyft/incubator-superset that referenced this pull request Sep 21, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 11, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 12, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 12, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 12, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
youngyjd pushed a commit to lyft/incubator-superset that referenced this pull request Oct 17, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
youngyjd pushed a commit to lyft/incubator-superset that referenced this pull request Oct 17, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
youngyjd pushed a commit to lyft/incubator-superset that referenced this pull request Oct 17, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
youngyjd pushed a commit to lyft/incubator-superset that referenced this pull request Oct 17, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
mistercrunch added a commit to lyft/incubator-superset that referenced this pull request Oct 29, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 30, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 30, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 30, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 30, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
betodealmeida pushed a commit to lyft/incubator-superset that referenced this pull request Oct 30, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
youngyjd pushed a commit to lyft/incubator-superset that referenced this pull request Nov 2, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
youngyjd pushed a commit to lyft/incubator-superset that referenced this pull request Nov 2, 2018
* [explore flow] handling duplicated column aliases

closes apache#5739

* lint

(cherry picked from commit 75bc501)
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0 labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Sqllab] Error: no such column When there are same fields in two tables with a join
3 participants