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

Download: Add link for Horizon Store app on stable release #937

Merged

Conversation

akien-mga
Copy link
Member

Depends on #936.

To merge only when we release 4.4-stable.

It adds a link to the /download/android page which is the one used for the current stable release. Currently it's 4.3 and the Horizon Store app is 4.4.dev2.

image

@akien-mga akien-mga added enhancement topic:content Issues and PRs related to static content labels Sep 23, 2024
@akien-mga akien-mga force-pushed the download-android-horizonstore branch 2 times, most recently from bbcf941 to 001555f Compare September 23, 2024 15:01
@akien-mga akien-mga force-pushed the master branch 2 times, most recently from e99f9b8 to 7bc695b Compare September 23, 2024 15:13
@coppolaemilio
Copy link
Member

What's the status on this one? should we salvage it?

@akien-mga akien-mga force-pushed the download-android-horizonstore branch from 001555f to 6fa6be7 Compare March 3, 2025 15:59
@akien-mga akien-mga marked this pull request as ready for review March 3, 2025 15:59
@akien-mga
Copy link
Member Author

Rebased and tested (on top of #1023) and it seems to work fine.

Currently it shows our popup asking for donations before going to the Meta Store in the same tab, so the popup is only visible for a fraction of a second. We might want to tweak things so such external links get opened in a new tab maybe.

But that shouldn't be blocking for this.

@akien-mga akien-mga requested a review from coppolaemilio March 3, 2025 16:03
@coppolaemilio
Copy link
Member

yeah, I can fix the popup in a followup PR

Copy link
Member

@coppolaemilio coppolaemilio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@akien-mga akien-mga merged commit b490db1 into godotengine:master Mar 3, 2025
1 check passed
@akien-mga akien-mga deleted the download-android-horizonstore branch March 3, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement topic:content Issues and PRs related to static content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants