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

Timeline Vision Data Issue fix #7230

Merged
merged 4 commits into from
Aug 14, 2020
Merged

Timeline Vision Data Issue fix #7230

merged 4 commits into from
Aug 14, 2020

Conversation

molecula451
Copy link
Contributor

@molecula451 molecula451 commented Aug 13, 2020

Description

Fix for the timeline vision timeline

vision

Refers/Fixes

Fixes #7237

Testing

@codecov
Copy link

codecov bot commented Aug 13, 2020

Codecov Report

Merging #7230 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7230      +/-   ##
==========================================
+ Coverage   26.10%   26.12%   +0.02%     
==========================================
  Files         297      297              
  Lines       29510    29508       -2     
  Branches     4377     4377              
==========================================
+ Hits         7703     7709       +6     
+ Misses      21536    21528       -8     
  Partials      271      271              
Impacted Files Coverage Δ
app/retail/views.py 20.85% <0.00%> (+0.08%) ⬆️
app/dashboard/embed.py 31.60% <0.00%> (+3.44%) ⬆️

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 b3cd45b...03b761b. Read the comment docs.

@molecula451 molecula451 changed the title tiny fix typo Timeline Data Issue fix Aug 13, 2020
@molecula451 molecula451 changed the title Timeline Data Issue fix Timeline Vision Data Issue fix Aug 13, 2020
@thelostone-mc
Copy link
Member

nice catch @molecula451

@thelostone-mc thelostone-mc merged commit a892f10 into gitcoinco:master Aug 14, 2020
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.

Issue on Timeline Vision page
2 participants