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

Hide non-curated non-Flatpak #1075

Closed
wants to merge 4 commits into from
Closed

Conversation

cassidyjames
Copy link
Contributor

Super experimental and probably heavy-handed, but it was something @danrabbit and I had talked about trying out locally on our installs.

The goal of this branch is to hide any non-curated non-Flatpak apps, making AppCenter more of a curated + Flatpak store. This should probably be done lower-level than where I'm doing it, though?

@davidmhewitt
Copy link
Member

If we are going to do this, then this probably isn't a great approach. We can save considerable amounts of processing power and time if we never construct these package rows in the first place.

We need to be careful to not exclude any OS updates and we need some thought about whether to still include non-curated, non-flatpak apps that are installed (i.e. will AppCenter still assist with updating all the apps on a user's PC?)

@cassidyjames
Copy link
Contributor Author

cassidyjames commented Sep 20, 2019

@davidmhewitt yeah that's kind of what I figured regarding just throwing the rows away; it felt like an unnecessary hammer but I mostly wanted to test the UX of not having those apps. 😁

I think it is reasonable to update all apps, but not display them in browsing/searching contexts. I could probably be convinced either way, though.

@danirabbit
Copy link
Member

I'm gonna close this since it's old and conflicts etc, but I opened #1432 to track the idea

@danirabbit danirabbit closed this Oct 13, 2020
@danirabbit danirabbit deleted the hide-uncurated-packagekit branch October 13, 2020 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants