Carnival

[j0ker1234]

Description

Checklist

• Read and conforms to the contributor guidelines
• Read and conforms to the style guidelines
• Linter status: 100% pass
• Added relevant tests
• Tests pass and code coverage has not decreased
• Changes don't break existing behavior
• Commit message follows commit guidelines
• Code documentation
• Completed: Code review by core team and any requested changes

Affected core subsystem(s)

Refers/Fixes

Testing and Sign-off

Contributor

• Read and followed the Contributor Guidelines
• Tested all changes locally
• Verified existing functionality
• Ran make test and everything passed!

Reviewer

• Affirm contributor guidelines have been followed and requested changes made
• CI tests and linting pass
• No conflicts (migrations, files, etc)
• Regression tested against staging or local deployment

Funder

• Validated requested changes were made to specification
• Bounty payout released to the contributor

[j0ker1234]

[codecov]

Codecov Report

Merging #3733 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3733   +/-   ##
=======================================
  Coverage   29.54%   29.54%           
=======================================
  Files         199      199           
  Lines       15332    15332           
  Branches     2006     2006           
=======================================
  Hits         4530     4530           
  Misses      10650    10650           
  Partials      152      152

Impacted Files	Coverage Δ
app/avatar/utils.py	14.8% <ø> (ø)	⬆️

---

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 5e565f5...70ff6ad. Read the comment docs.

[thelostone-mc]

@j0ker1234 that looks pretty dope ! can you resolve the conflicts though ?

[thelostone-mc]

@j0ker1234 wanna get this in this week ?

[j0ker1234]

@thelostone-mc I am working on some more designs, will submit a pr by Friday.

[j0ker1234]

@thelostone-mc I have resolved the conflicts for this branch. For the other designs will create a new pr.

[thelostone-mc]

@j0ker1234 Ah merging your other PR introduced conflicts here again 😅 Mind resolving them again?

[j0ker1234]

@thelostone-mc Done.