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

POC - bufficorn avatar builder for ETHDenver #5621

Merged
merged 1 commit into from
Dec 11, 2019

Conversation

owocki
Copy link
Contributor

@owocki owocki commented Dec 11, 2019

bufficorn avatar builder for ETHDenver

TODO: make it so that the gitcoin platform handles multiple 3d avatar types..

need to do some work around making this a parameter

Screen+Recording+2019-12-10+at+07 57+PM

@codecov
Copy link

codecov bot commented Dec 11, 2019

Codecov Report

Merging #5621 into master will not change coverage.
The diff coverage is 40%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5621   +/-   ##
=======================================
  Coverage   30.22%   30.22%           
=======================================
  Files         248      248           
  Lines       21186    21186           
  Branches     3068     3068           
=======================================
  Hits         6403     6403           
  Misses      14507    14507           
  Partials      276      276
Impacted Files Coverage Δ
app/avatar/views_3d.py 18.48% <40%> (ø) ⬆️

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

@danlipert
Copy link
Contributor

Rad!

@danlipert danlipert merged commit 5654ad6 into master Dec 11, 2019
danlipert added a commit that referenced this pull request Dec 11, 2019
danlipert added a commit that referenced this pull request Dec 11, 2019
@danlipert
Copy link
Contributor

@owocki During pre-deploy testing I discovered some issues with the avatar builder - theres some bug thats also on production that puts you on an odd looking onboarding screen once you save your avatar, but also the Bufficorn avatar that gets saved looks pretty jacked up. Check it out and let me know what you think

@owocki
Copy link
Contributor Author

owocki commented Dec 11, 2019

shoot sorry - this was just a POC not meant to go live.. should have tagged it as such

@owocki owocki changed the title bufficorn avatar builder for ETHDenver POC - bufficorn avatar builder for ETHDenver Dec 11, 2019
@thelostone-mc thelostone-mc deleted the kevin/bufficorn_avatar_builder branch June 27, 2020 00:49
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