-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Change conditions for being indicated "new contributor" #3865
Comments
I want to work on this |
oh that would be great!
…On Thu, Nov 1, 2018 at 7:29 AM narnt ***@***.***> wrote:
I want to work on this
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3865 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJ4Lr5hr6tWgCLPs5IEG_p6jOPqq9ks5uqtr0gaJpZM4YEHYR>
.
|
Hi @narnt -- have you had any luck with this so far? If you want a partner or have moved on, I'd love to step in and assist you/take the reins, as appropriate. Or if the issue is solved and ready to be closed, that's helpful information, too! Thanks! |
Surely would want to work, if any problems are faced, I would love to help @narnt...Thanks! |
hi @quinn-codes-synthesis and @coderjolly ! thank you for your offers of assistance! I just have had a break, but I have struggled with anoter issue: #3707 |
Hi @jywarren , I need help with my PR! |
Pls tell @narnt, I will try to help u out |
@coderjolly thank you! Link to my PR is above. There are some issues: https://codeclimate.com/github/publiclab/plots2/pull/3928 How can I resolve it? |
Hi! OK, i'll reply back in the PR! |
plots2/app/models/node_tag.rb
Line 24 in c7f2ce8
Here,
if Node.where(:uid => @uid).length === 1
determines if you're a new contributor, which is not quite right -- someone may have contributed only one node, but they might've done it two years ago, and sonew contributor
perhaps needs an additional condition.What about if it's their first post, AND the post is less than 1 month old? @publiclab/community-reps ?
This code appears in 2 places (and in #3864 we may be trying to consolidate these):
plots2/app/models/node_tag.rb
Lines 22 to 25 in af295d7
plots2/app/models/user.rb
Lines 53 to 56 in 4f028aa
The text was updated successfully, but these errors were encountered: