Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

[FEATURE] Popular profile page to show random users at the top #2558

Closed
eddiejaoude opened this issue Dec 31, 2022 · 9 comments · Fixed by #2803
Closed

[FEATURE] Popular profile page to show random users at the top #2558

eddiejaoude opened this issue Dec 31, 2022 · 9 comments · Fixed by #2803
Assignees
Labels

Comments

@eddiejaoude
Copy link
Member

Description

Maybe select 2 random users to appear at the top of the popular profile page to help create awareness of profiles

Screenshots

No response

Additional information

No response

@github-actions
Copy link
Contributor

It's great having you contribute to this project

Welcome to the community 🤓

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

@thevinitgupta
Copy link
Contributor

How would the random users be selected?

@eddiejaoude
Copy link
Member Author

What do you mean? It will be selected at random from the list of users in the database

@thevinitgupta
Copy link
Contributor

I wanted to know if the selection will be completely random or any fixed algorithm to select.

@eddiejaoude
Copy link
Member Author

Let's keep it simple and make it completely random

@ManishBisht777
Copy link
Contributor

@eddiejaoude can we make a button to switch between popular/featured profiles and random/new users by default we can show new/random users
also by this so we don't have to make a separate API call for just 2-3 user :)

@eddiejaoude
Copy link
Member Author

I think as we have small cards, having 2 sections on a single page would be better for now - the API calls are done on the server side I think

@eddiejaoude
Copy link
Member Author

@ManishBisht777 there is still only 1 API call, as the end point is now discover not popular, so it will return all the required data 👍

@ManishBisht777
Copy link
Contributor

@ManishBisht777 there is still only 1 API call, as the end point is now discover not popular, so it will return all the required data +1

That's nice

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants