Skip to content
This repository has been archived by the owner on Sep 3, 2023. It is now read-only.

Changed logic regarding minDate and maxDate in dateTime module to allow currentDate to be equal to one of them #2066

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

peter-witt
Copy link

Guidelines

  1. You must run the spotlessApply task before commiting, either through Android Studio or with ./gradlew spotlessApply.
  2. A PR should be focused and contained. If you are changing multiple unrelated things, they should be in separate PRs.
  3. A PR should fix a bug or solve a problem - something that only you would use is not necessarily something that should be published.
  4. Give your PR a detailed title and description - look over your code one last time before actually creating the PR. Give it a self-review.

If you do not follow the guidelines, your PR will be rejected.

PR Description

Changed logic regarding minDate and maxDate in dateTime module to allow currentDate to be equal to one of them.
This is to make it possible to have two dates as allowed dates, and preselect one of them, which was not possible before. The user was able to select one of the two dates, but the code was not allowing preselection of the dates. For example with these dates:

minDate = 2021.01.01
maxDate = 2021.01.02
currentDate = 2021.01.01

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant