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

[4.0.0-beta.18] A non-existing date can be selected #16733

Closed
duyetnk opened this issue Dec 14, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@duyetnk
Copy link

commented Dec 14, 2018

Bug Report

Ionic version:
[x] 4.x

Current behavior:
You can select a non-existing date:
ion-datetime-wrongdate
ion-datetime_wrongdate 2

Expected behavior:

All non-existing dates will not show when picking months

Related code:

<ion-item>
	<ion-label>Select Date</ion-label>
	<ion-datetime displayFormat="MM/DD/YYYY" [(ngModel)]="selectDate"  min="2018" max="2023"></ion-datetime>
</ion-item>

Other information:

Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.5.0 (C:\Users\naugh\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.18
   @angular-devkit/build-angular : 0.10.5
   @angular-devkit/schematics    : 0.7.5
   @angular/cli                  : 7.0.3
   @ionic/angular-toolkit        : 1.2.0

Cordova:

   cordova (Cordova CLI) : 8.1.2 (cordova-lib@8.1.1)
   Cordova Platforms     : android 7.1.4
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.2.5, (and 13 other plugins)

System:

   NodeJS : v10.13.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.4.1
   OS     : Windows 10

@ionitron-bot ionitron-bot bot added the triage label Dec 14, 2018

@duyetnk duyetnk changed the title [4.0.0-beta18] Can select a non-existing date [4.0.0-beta.18] Can select a non-existing date Dec 14, 2018

@duyetnk duyetnk changed the title [4.0.0-beta.18] Can select a non-existing date [4.0.0-beta.18] A non-existing date can be selected Dec 14, 2018

@liamdebeasi

This comment has been minimized.

Copy link
Member

commented Feb 12, 2019

Hi there,

Thanks for the issue! I’ve added this to issue #16630 to keep track of. We’re going to close out the individual issues and address all of the Datetime issues at once. We’re not forgetting about your issue, just trying to keep them all together.

Thanks for using Ionic!

@ionitron-bot

This comment has been minimized.

Copy link

commented Mar 14, 2019

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Mar 14, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.