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

OpenSourceDonation: Set default time value on page load in case user skips the page #1276

Conversation

sunkup
Copy link
Member

@sunkup sunkup commented Jan 29, 2025

Purpose

We are not setting any default time even though it shows the first option as selected. Now when the user skips the page, it shows up repeatedly until a time is actually tapped/selected.

Short description

  • Set default time value on page load in case user skips the page

Checklist

  • The PR has a proper title, description and label.
  • I have self-reviewed the PR.
  • I have added documentation to complex functions and functions that can be used by other modules.
  • I have added reasonable tests or consciously decided to not add tests.

@rfc2822
Copy link
Member

rfc2822 commented Feb 1, 2025

For an alternative suggestion, see #1287. What do you think?

@rfc2822
Copy link
Member

rfc2822 commented Feb 2, 2025

Closed in favor #1287

@rfc2822 rfc2822 closed this Feb 2, 2025
@rfc2822 rfc2822 deleted the 1275-open-source-intro-page-default-dont-remind-for-1-month-does-nothing branch February 2, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Open-source intro page default "don't remind for 1 month" does nothing
2 participants