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

[WIP] Visualize after running Ctas query (#4) #6

Closed
wants to merge 4 commits into from

Conversation

bkyryliuk
Copy link
Owner

  • Visualize after running Ctas query.

Stub for table viz

Work on the viz

Add JS functionality for Ctas viz

Add test for the table viz flow

Fix test

Add js test for the ctas viz

  • Fix tests

CATEGORY

Choose one

  • Bug Fix
  • Enhancement (new features, refinement)
  • Refactor
  • Add tests
  • Build / Development Environment
  • Documentation

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

REVIEWERS

@bkyryliuk bkyryliuk force-pushed the bogdan/cta_viz branch 5 times, most recently from fa4e76f to 21b010c Compare March 24, 2020 18:14
@codecov-io
Copy link

codecov-io commented Mar 24, 2020

Codecov Report

Merging #6 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   58.81%   58.81%           
=======================================
  Files         379      379           
  Lines       12209    12209           
  Branches     3007     3007           
=======================================
  Hits         7181     7181           
  Misses       4848     4848           
  Partials      180      180           

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 e4d7673...e4d7673. Read the comment docs.

@bkyryliuk bkyryliuk force-pushed the bogdan/cta_viz branch 7 times, most recently from 8af8ccf to 1484bb4 Compare March 27, 2020 23:50
bkyryliuk and others added 3 commits April 9, 2020 20:25
* Visualize after running Ctas query.

Stub for table viz

Work on the viz

Add JS functionality for Ctas viz

Add test for the table viz flow

Fix test

Add js test for the ctas viz

* Fix tests
@bkyryliuk bkyryliuk force-pushed the bogdan/cta_viz branch 2 times, most recently from d8cd1c2 to 720fb22 Compare April 10, 2020 17:05
@bkyryliuk bkyryliuk force-pushed the master branch 2 times, most recently from fb0a0d3 to 7b11b44 Compare April 16, 2020 17:09
bkyryliuk added a commit that referenced this pull request Apr 16, 2020
…#6)

* Create additional endpoint that vizualized the new table

* Unit tests

* Use proper schema depending on the engine
bkyryliuk added a commit that referenced this pull request May 13, 2020
…#6)

* Create additional endpoint that vizualized the new table

* Unit tests

* Use proper schema depending on the engine
@bkyryliuk bkyryliuk closed this Jun 17, 2020
bkyryliuk added a commit that referenced this pull request Jun 23, 2020
…#6)

* Create additional endpoint that vizualized the new table

* Unit tests

* Use proper schema depending on the engine
bkyryliuk added a commit that referenced this pull request Jun 25, 2020
…#6)

* Create additional endpoint that vizualized the new table

* Unit tests

* Use proper schema depending on the engine
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.

3 participants