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

Add link to donation page on app drawer #11870

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from
Draft

Conversation

theScrabi
Copy link
Member

@theScrabi theScrabi commented Jan 2, 2025

At 38c3 we discussed how we could make the request for donations more visible. We discussed multiple models. Ultimately we could not agree on a good solution to actually ask the user for donations.
However, we all agreed that donations should be more visible than just show up in the about screen.

This will add the link for donations to the app drawer. When clicking it will open the donation page of the NewPipe website.

Screenshot_1735821499

@github-actions github-actions bot added the size/small PRs with less than 50 changed lines label Jan 2, 2025
Comment on lines +937 to +941
private void openDonationPage() {
final Intent intent = new Intent(Intent.ACTION_VIEW,
Uri.parse(getString(R.string.donation_url)));
startActivity(intent);
}
Copy link
Member

Choose a reason for hiding this comment

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

Use ShareUtils to create and start the intent.

@AudricV AudricV changed the title add donation link to app drawer Add link to donation page on app drawer Jan 2, 2025
@AudricV AudricV added the template missing The bug/feature template is missing (e.g. the used app does not support issue templates) label Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/small PRs with less than 50 changed lines template missing The bug/feature template is missing (e.g. the used app does not support issue templates)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants