ryo-ma / github-profile-trophy

🏆 Add dynamically generated GitHub Stat Trophies on your readme
https://github-profile-trophy.vercel.app
MIT License
5.18k stars 767 forks source link

Improve Requirements for Organization Secret Trophy #63

Closed rayhanadev closed 7 months ago

rayhanadev commented 3 years ago

After finding out that the new secret trophy requires the person to be in [REDACTED] organizations, the first reaction would be to make [REDACTED] organizations to get it. Unsurprisingly this works, however it doesn't seem right. Instead there should be checking and requirements on the organizations before awarding the trophy:

Proposals:

  1. The organization must be (at least) a week old
  2. The organization must contain more than one account
  3. The organization must contain one or more repositories
  4. The person should be in at least one organization that was not created by them* **

* I believe this one might be controversial. Any thoughts? ** Dropped this

ryo-ma commented 3 years ago

I got your feelings. Regarding 2. and 3., I think it a good idea to add to the rules. I think 1. has little effect on the problem. I couldn't understand the meaning of 4. well.

Alternatively, there may be a way to add value by applying rule 2. to create a new trophy. ex. Big Org Owner: The organization must contain than ten accounts and, must be the owner of its organization.

rayhanadev commented 3 years ago

@ryo-ma gotcha, if you want I can open a PR adding in 2. and 3. to the current trophy.

I couldn't understand the meaning of 4. well.

So if a user is in three public organizations and they created (i.e. they are the owner of) two organizations then the third one must be an outside organization they are a member of in order to get the trophy. It kinda makes sure they didn't just create three organizations and add fake accounts to it. More credibility.

Alternatively, there may be a way to add value by applying rule 2. to create a new trophy. ex. Big Org Owner: The organization must contain than ten accounts and, must be the owner of its organization.

I think there are a fair bit too many secret trophies 😅. Maybe we can make this one a ranked trophy starting at an organization with 2 accounts and working up through the ranks?

spenserblack commented 3 years ago

I understand the reasons for 1. and 4., but here are my thoughts on it:

1.

This just means the user has to wait a week to get the trophy.


4.

It kinda makes sure they didn't just create three organizations and add fake accounts to it.

If I have real account A and fake account B, then creating 2 organizations with A and 1 with B would pass this requirement, wouldn't it?

I also think that 4. would be biased against some very important contributors. The meaning of creating an organization and joining an organization are different. To create an organization is to be a leader in a project IMO. To join an organization is to be a very important contributor, but not necessarily the leader/creator of something. Is it fair that someone who is always the leader wouldn't get the trophy?

rayhanadev commented 3 years ago

@spenserblack

This just means the user has to wait a week to get the trophy.

Fair enough, but more "secure" is better than not secure at all.

I also think that 4. would be biased against some very important contributors.

Yeah that's what I was running in the back of my mind and I do agree with you. I'll drop 4. from the list then.

ryo-ma commented 3 years ago

@RayhanADev I got it. If you make a pull request about 2. and 3., I will accept it.

ryo-ma commented 3 years ago

@Time-dragon Can you explain why it's a bad idea?

ryo-ma commented 3 years ago

you get tired of making more on your account so you resort to organizations.

I don't understand the situation.

It is officially stated that the GitHub Organization is aimed at collaboration. https://docs.github.com/en/organizations/collaborating-with-groups-in-organizations/about-organizations

ryo-ma commented 3 years ago

@RayhanADev After all, do you make a pull request on this issue?

rayhanadev commented 3 years ago

@ryo-ma yeah I’m currently moving to a different city right now so I’m on my phone, not my computer. I’ll have it setup tomorrow and I can continue what I was working on but if that’s too late feel free to go ahead and PR yourself.

Sorry about this, I tried to finish ASAP but the movers came much earlier than expected!

ryo-ma commented 3 years ago

@RayhanADev It's not particularly urgent, so it's okay at your own pace. Living in a different city is exciting!

github-actions[bot] commented 9 months ago

This issue is stale because it has been open for 30 days with no activity.

D3vil0p3r commented 9 months ago

Currently is there a chance to add in "Star Trophy" also the stars coming from the org I am the owner? If so, how?

github-actions[bot] commented 8 months ago

This issue is stale because it has been open for 30 days with no activity.

D3vil0p3r commented 8 months ago

Still relevant.

github-actions[bot] commented 7 months ago

This issue is stale because it has been open for 30 days with no activity.