-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
Propose work #4478
Propose work #4478
Conversation
app/dashboard/views.py
Outdated
profile_json['position_contributor'] = user.get_contributor_leaderboard_index() | ||
profile_json['position_funder'] = user.get_funder_leaderboard_index() | ||
profile_json['work_done'] = count_work_completed | ||
profile_json['work_inprogress'] = count_work_in_progress | ||
|
||
profile_json['job_status'] = user.job_status_verbose if user.job_search_status else None | ||
profile_json['verification'] = user.get_my_verified_check | ||
profile_json['avg_rating'] = user.get_average_star_rating |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
^ would it be cleaner to redefine the whole bit as
profile_json = {
id: user.id,
...
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thelostone-mc actually is done here #4477
Codecov Report
@@ Coverage Diff @@
## master #4478 +/- ##
==========================================
+ Coverage 30.08% 30.15% +0.06%
==========================================
Files 209 208 -1
Lines 16850 16821 -29
Branches 2267 2259 -8
==========================================
+ Hits 5070 5072 +2
+ Misses 11582 11557 -25
+ Partials 198 192 -6
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #4478 +/- ##
==========================================
- Coverage 30.09% 30.07% -0.03%
==========================================
Files 209 209
Lines 16848 16860 +12
Branches 2266 2270 +4
==========================================
Hits 5070 5070
- Misses 11580 11592 +12
Partials 198 198
Continue to review full report at Codecov.
|
Description
Refers/Fixes
#4416
Testing