-
-
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
Tribes light housekeeping #7143 #7171
Conversation
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.
- I'm not sure about lodash being added just for one usage! @octavioamu what do you think?
- Could we have video recording/ screenshots of what these changes are so that the product team can verify them
I added loadash so it would be easy to add more functionalities for filters later on too. |
cool we are in general very strict with new libs additions but as loadsh is not more than 72kb I think is fine. |
please after you add the video change it to ready to review so we can check it. The code looks fine. |
6ba780d
to
505c170
Compare
Codecov Report
@@ Coverage Diff @@
## master #7171 +/- ##
==========================================
+ Coverage 18.08% 26.18% +8.09%
==========================================
Files 285 297 +12
Lines 29393 29385 -8
Branches 4366 4342 -24
==========================================
+ Hits 5316 7694 +2378
+ Misses 24065 21415 -2650
- Partials 12 276 +264
Continue to review full report at Codecov.
|
@octavioamu @PixelantDesign Here's the walkthrough recording -> https://www.loom.com/share/7b55e1998cd84e579b7fb7e433e333d5 |
Features which this pr doesn't address:-
|
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.
Looking good! @androolloyd are you able to help with the performance piece?
d5df988
to
08725cb
Compare
@sanchaymittal Thanks for the PR - looks like there are a lot of automatic changes from your IDE where the files are being autoformatted. Can you please limit your changes to the relevant code for this issue and remove any of the changes that are style related? Generally we use eslint so as long as that passes then its good. |
Current bounties tab was only viewable if user follows the tribe. Change bounties tab is default for tribe.
before: All people view section under tribe sorted based on user id. change: Sort all people view on the basis of users contributor rank.
user story: Help sponsors troubleshoot If I’m an admin of gitcoin, I should also see admin view of tribes (so core team can help)
This reverts commit d989524822245e3e090f60bb63e3608b39d0155c.
user story: Help sponsors troubleshoot If I’m an admin of gitcoin, I should also see admin view of tribes (so core team can help)
- It shows follow suggestion for top member based on contributor rank. - Once the user is followed it will be disappear from the list - Top 10 profiles will be shown based on the contributor rank. - Ignore the current user profile in townsquare suggestion bar. - remove unwanted changes from prior commit in user.js
08725cb
to
77330fe
Compare
yayyyy |
Description
If I’m an admin of gitcoin, I should also see admin view of tribes (so core team can help)
Refers/Fixes
#7143
Testing