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

Check for previous date set for reminder #991

Open
pratistha-05 opened this issue Sep 15, 2024 · 5 comments
Open

Check for previous date set for reminder #991

pratistha-05 opened this issue Sep 15, 2024 · 5 comments
Assignees

Comments

@pratistha-05
Copy link

When we set reminder, there is no check to not allow user to select past time or dates in date picker. Even after selecting wrong date, the reminder is set without any check to notify user to change the date. Possible solutions can be:

  1. Provide restriction to select previous date in date picker
  2. Show toast message to user to select appropriate dates
  3. show snackbar to let user know reminder cannot be set
@eren8204
Copy link
Contributor

eren8204 commented Oct 3, 2024

@pratistha-05 can i start working on it

@eren8204
Copy link
Contributor

eren8204 commented Oct 3, 2024

@federicoiosue hi i have added the functionality of preventing past reminder, can i make pull request now?

@eren8204
Copy link
Contributor

eren8204 commented Oct 3, 2024

@pratistha-05 @federicoiosue add me as collaborator so that i can contribute

@federicoiosue
Copy link
Owner

@eren8204 you can open a pull request to be reviewed with no permission requirements on the project itself.

@eren8204
Copy link
Contributor

eren8204 commented Oct 5, 2024

@federicoiosue i've raised the PR please look into that

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

No branches or pull requests

3 participants