Skip to content

Conversation

@danlipert
Copy link
Contributor

Description

This PR fixes an issue where a username starting with the prefix "web3" would get handled by the wrong view if trying to view their profile.

Refers/Fixes

reported on slack

Testing

tested locally

@codecov
Copy link

codecov bot commented Apr 30, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6535   +/-   ##
=======================================
  Coverage   27.26%   27.26%           
=======================================
  Files         287      287           
  Lines       26712    26712           
  Branches     3945     3945           
=======================================
  Hits         7284     7284           
+ Misses      19161    19157    -4     
- Partials      267      271    +4     
Impacted Files Coverage Δ
app/app/urls.py 86.20% <ø> (ø)
app/dashboard/views.py 11.12% <0.00%> (ø)
...rketing/management/commands/no_applicants_email.py 0.00% <0.00%> (ø)
...eting/management/commands/assemble_leaderboards.py 40.48% <0.00%> (ø)

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 21a70e3...9bae590. Read the comment docs.

@thelostone-mc thelostone-mc merged commit 87f999a into master Apr 30, 2020
@octavioamu octavioamu mentioned this pull request Apr 30, 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.

3 participants