Improvement on Top Downloaded Apps

[yihangx]

Now, top downloaded apps show all-time top downloaded apps, which is kind of outdated. It should show top downloaded apps in last 24 months.

Related code:

appstore/apps/views.py

Line 90 in dce25f1

downloaded_apps = App.objects.filter(active=True).order_by('downloads').reverse()[:_DefaultConfig.num_of_top_apps]

appstore/apps/views.py

Line 118 in dce25f1

def all_apps_downloads(request):

[qzMalekuz]

Hi @yihangx ,

I reviewed the download tracking models and noticed that historical download data is stored per release via ReleaseDownloadsByDate.

My understanding is that implementing this would involve:

Filtering ReleaseDownloadsByDate records for the last 24 months

Aggregating count grouped by App (via the Release relationship)

Ordering apps by the aggregated value

Before proceeding, I’d like to confirm:

Should this logic replace the existing downloads field for the “Top Downloaded Apps” section only?

Or should the downloads field itself be recalculated dynamically?

I can prepare a PR using ORM aggregation once the intended behavior is confirmed.

Thanks.

[coleslaw481]

Hi @yihangx ,

I reviewed the download tracking models and noticed that historical download data is stored per release via ReleaseDownloadsByDate.

My understanding is that implementing this would involve:

Filtering ReleaseDownloadsByDate records for the last 24 months

Aggregating count grouped by App (via the Release relationship)

Ordering apps by the aggregated value

Before proceeding, I’d like to confirm:

Should this logic replace the existing downloads field for the “Top Downloaded Apps” section only?

Or should the downloads field itself be recalculated dynamically?

I can prepare a PR using ORM aggregation once the intended behavior is confirmed.

Thanks.

Yes it should only be used to show what apps are displayed in the top downloaded apps UI.

thanks,
chris

[qzMalekuz]

Hi @coleslaw481 ,

I’ve opened a PR - #119 implementing the 24-month aggregation logic for
Top Downloaded Apps UI using ReleaseDownloadsByDate.

Please let me know if you’d like any adjustments.

Thanks!