Skip to content

Generate MOST_POPULAR_APPS based on catalog contents - #10

Open
orhtej2 wants to merge 5 commits into
YunoHost:mainfrom
orhtej2:generate_most_popular_apps
Open

Generate MOST_POPULAR_APPS based on catalog contents#10
orhtej2 wants to merge 5 commits into
YunoHost:mainfrom
orhtej2:generate_most_popular_apps

Conversation

@orhtej2

@orhtej2 orhtej2 commented Mar 7, 2026

Copy link
Copy Markdown
Contributor

Fetch the catalog and stars and zip these together for a list of 29 real apps + example_ynh

Comment thread server.py Outdated
Comment thread server.py
Comment thread server.py Outdated
@tituspijean
tituspijean force-pushed the generate_most_popular_apps branch from 068511e to 608d8e7 Compare July 24, 2026 20:49
@tituspijean
tituspijean force-pushed the generate_most_popular_apps branch from f741d0e to 86a4197 Compare July 25, 2026 08:08
@tituspijean

This comment was marked as resolved.

@tituspijean
tituspijean force-pushed the generate_most_popular_apps branch from 0b9e0cf to b8000dd Compare July 25, 2026 08:10
@tituspijean
tituspijean force-pushed the generate_most_popular_apps branch from b8000dd to 1b3d891 Compare July 25, 2026 08:21
@tituspijean

tituspijean commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

I have just tested this on the infra: somehow the list is only ['example'].

It looks like read_app_list() does not return anything.

@orhtej2

orhtej2 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

@tituspijean fixed by applying different instances of TTLCache to cached properties.

We could in fact get away with a single one on MOST_POPULAR_APPS, WDYT?

@orhtej2

orhtej2 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

(adjusted as per comment, only the list is now cached)

@tituspijean

Copy link
Copy Markdown
Contributor

Tested on production, looks good to go.

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.

2 participants