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

Better experience when many projects are available to import #13

Closed
takluyver opened this issue Jan 6, 2016 · 15 comments
Closed

Better experience when many projects are available to import #13

takluyver opened this issue Jan 6, 2016 · 15 comments

Comments

@takluyver
Copy link

I like the 'import project' design introduced in readthedocs/readthedocs.org#1695 overall, but the pagination is annoying me. I'm in several organisations with many repositories, and as my username is relatively late in the alphabet, I have to click through many pages to get to my own personal repos.

How could this be better:

  • I can filter down to an organisation's repos easily, but not to my own personal repos (takluyver/foo). I guess this would be simple to add.
  • Put a search box on the import project page, preferably with search-as-you-type.
  • Sort recently updated repos to the top, like Github's repo view, as they're the ones I'm most likely to want to use. I'm baffled by the sort order at the moment - it seems mostly alphabetical, with some things scattered randomly into it.
@agjohnson
Copy link
Contributor

Yes, sort is alphabetical, with grouping by organization. This pagination is a first pass, the second pass is to implement a fuzzy search, keeping pagination. We can't sort on most used from github, we don't have the information to make that decision.

@agjohnson
Copy link
Contributor

For anyone willing to take this work on, it should be a mostly digestible chunk. The front end interaction is entirely tied to the API results and pagination. So we'll want to offer fuzzy search through the API. Pagination will stay the same, but will also operate on a filtered list of elements. This would require few changes on the front end side, minus the addition of a search box that triggers a fuzzy search on after a delay from typing or something.

@adrn
Copy link

adrn commented Aug 10, 2017

+1 to this

A minimal addition to get all repos that don't belong to an organization would be incredibly helpful. I'm currently in a state where I can't use the automatic import because I'm in the conda-forge organization and I have 10s of pages of repos that make finding my own projects tedious and impractical.

@humitos
Copy link
Member

humitos commented Dec 28, 2017

Related to readthedocs/readthedocs.org#3337

@ericholscher
Copy link
Member

Seems with a lot of repos, this is now timing out, so something we should re-prioritize.

@agjohnson
Copy link
Contributor

What specifically is timing out?

That might be something to address separately if it is a problem with data modeling, as this is discussing the UI interaction. If it's a matter of our indexes being poorly defined for how we are querying, then adding fuzzy search could turn this slow query into a 100% unusable query. I'd rather go the route of fuzzy search for our v2 of this feature. Perhaps we disregard the need for pagination for our next implementation -- especially if it adds to query speed slow down, as pagination is always going to be slow.

On the data modeling side we also have an issue open somewhere about redefining this modeling, so that RemoteRepository instances are defined once, and have a many-to-many relation to the remote user instance. So, instead of there being a RemoteRepository for rtfd/readthedocs.org repository for everyone in our org, there is one RemoteRepository instance that is shared. This reduces table bloat, but also makes managing webhooks make more sense.

@wolph
Copy link

wolph commented Mar 18, 2019

The timing out is now fixed luckily :)
readthedocs/readthedocs.org#5441

But it's definitely not ideal for people like me with too many repos. Even when just looking through my own repos I've got 14 pages to browse through. I think that even a simple repo_name__istartswith would be more than sufficient. Assuming that in the case of this repo it would search in the readthedocs.org portion and not the entire rtfd/readthedocs.org, not sure if that field is available.
A full icontains would most likely be very slow if you have the 6400 conda-forge packages in the list of available options.

@agjohnson agjohnson transferred this issue from readthedocs/readthedocs.org May 20, 2020
@blombergedvin
Copy link

Want to give this issue a bump :)
I believe we have at least 150 pages of projects in my organisation, and importing a project at the end takes more than 10 minutes of clicking next.

A search would be ideal (either a "begins with" or contain would work).
Another alternative would be to allow you to jump to a specific page in the pagination, either in the UI or reflect the current page in the URL.

@humitos
Copy link
Member

humitos commented Jun 29, 2023

@blombergedvin can you give it a try to our new beta dashboard at https://beta.readthedocs.org/ and let us know if the UX for importing a project has improved? 😄

@blombergedvin
Copy link

@humitos Exciting! I was not able to login with my normal username/email, even though the same works in the old dashboard.
Get a: Error The username and/or password you specified are not correct..

@humitos
Copy link
Member

humitos commented Jul 5, 2023

@blombergedvin that's pretty weird -- the db is exactly the same, so if you are able to login on readthedocs.org, you should be able to login with the same user/pass in beta.readthedocs.org

By the way, are you referring to the "old dashboard" to the community (readthedocs.org) or the business version (readthedocs.com)?

@blombergedvin
Copy link

@humitos That is probably the reason. I'm using the business version with https://readthedocs.com. Didn't notice the slight difference in the url.
Is there a beta dashboard for the business version as well?

@humitos
Copy link
Member

humitos commented Jul 5, 2023

Is there a beta dashboard for the business version as well?

Not yet, but we are working on it. Hopefully, we will publish a beta version for the business version soon.

@humitos
Copy link
Member

humitos commented Jun 5, 2024

For all of those that have hit this issue in the past, I want to let them know that we already released the new dashboard that solves this problem:

Let us know your experience with this pattern and if it works as you expected 👍🏼

@agjohnson
Copy link
Contributor

I think this issue can be closed, as the new dashboard resolves this. I'll removed this from the dashboard roadmap at least, as it's been long solved there.

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

No branches or pull requests

7 participants