Skip to content
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

Remove the unknown trophies (as don't look good) #250

Closed
wants to merge 2 commits into from

Conversation

bhavberi
Copy link
Contributor

@bhavberi bhavberi commented Dec 3, 2023

Removed the unknown ranked trophies from the permanent trophies, as they don't look good on a profile...

Copy link

vercel bot commented Dec 3, 2023

@bhavberi is attempting to deploy a commit to the ryo-ma's Team Team on Vercel.

A member of the Team first needs to authorize it.

@bhavberi
Copy link
Contributor Author

bhavberi commented Dec 3, 2023

@alexcastrodev Requesting your review and suggestions.

@alexcastrodev
Copy link
Collaborator

@ryo-ma introduces unknows, he knows better if makes sense.

In my oppinion for this PR, you are not removing, but filtering. if we need to remove ( i dont know if it is necessary ), we should remove permanently.
Also, missing tests.

@bhavberi
Copy link
Contributor Author

bhavberi commented Dec 5, 2023

Yeah, the trophies with 0 points (given the rank of Unknown) would not show on user profiles by default. Here, the meaning of remove meant from user badges.
Couldn't get what you mean by permanently?

@bhavberi
Copy link
Contributor Author

bhavberi commented Dec 5, 2023

Also, what kind of tests are missing?

@iamxerrycan
Copy link

how can i contribute , iam new to git

@iamxerrycan
Copy link

i am not getting

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