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

Unable to parse date range from input value #2871

Closed
hagen93 opened this issue Oct 31, 2023 · 2 comments · Fixed by #2876
Closed

Unable to parse date range from input value #2871

hagen93 opened this issue Oct 31, 2023 · 2 comments · Fixed by #2876

Comments

@hagen93
Copy link
Contributor

hagen93 commented Oct 31, 2023

Hi,

I was trying to initialize the picker from the input fields HTML value attribute, but I'm just ending up with erroneous single dates instead of ranges. Seems to me that the error might be in this line:

if (this.optionsStore.options.multipleDates) {
which probably should do the parsing for multiple in cases where dateRange is true too, e.g. condition might be this.optionsStore.options.multipleDates || this.optionsStore.options.dateRange.

I tried enabling multipleDates as a workaround, but that seems to be invalid configuration when dateRange is enabled.

@hagen93
Copy link
Contributor Author

hagen93 commented Oct 31, 2023

For reference my configuration is as follows:

{
    dateRange: true,
    multipleDatesSeparator: ' - ',
    display: {
        components: {
            clock: false
        }
    },
    localization: {
        format: 'dd/MM/yyyy'
    }
}

@Eonasdan Eonasdan moved this from 🆕 New to 👀 In review in Tempus Dominus Nov 6, 2023
@Eonasdan Eonasdan added the State: Pending Release This issue has been resolved in the dev branch and is waiting on a new release label Nov 6, 2023
@Eonasdan Eonasdan mentioned this issue Nov 10, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Tempus Dominus Nov 10, 2023
@Eonasdan Eonasdan removed the State: Pending Release This issue has been resolved in the dev branch and is waiting on a new release label Nov 10, 2023
@Eonasdan
Copy link
Owner

Hello. This was released with 6.7.19

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

Successfully merging a pull request may close this issue.

2 participants