-
Notifications
You must be signed in to change notification settings - Fork 104
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
feat(date): allow restrictions for dates and query date ranges #2646
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2646 +/- ##
============================================
+ Coverage 43.34% 43.59% +0.24%
- Complexity 887 905 +18
============================================
Files 78 78
Lines 3368 3386 +18
============================================
+ Hits 1460 1476 +16
- Misses 1908 1910 +2 🚀 New features to boost your workflow:
|
c610911
to
2fb862b
Compare
88a2440
to
231b87a
Compare
231b87a
to
db7bee7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice! :) Added some wording suggestions to make it a bit more easy to understand / less technical.
And I’d say the icons can be removed as they are the same and thus mostly visual noise? It’s a sub-item of the checkbox anyway so it’s fine to not have an icon and thus look a bit indented.
<template #icon> | ||
<Pencil :size="20" /> | ||
</template> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<template #icon> | |
<Pencil :size="20" /> | |
</template> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment below
<template #icon> | ||
<Pencil :size="20" /> | ||
</template> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<template #icon> | |
<Pencil :size="20" /> | |
</template> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment below
@jancborchardt thanks for the review :)
No, they are separate from the checkbox and always shown. The restrictions of which date can be chosen apply also to single date questions. That's why I added the icons. Without them it felt wrong to me. |
6eabe13
to
7b55891
Compare
This adds the possibility to add restrictions to the selectable date and to ask for a date range Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
7b55891
to
b2310a2
Compare
This closes #2396 by allowing to set minimum and maximum dates and to query date ranges
Set question settings for minimum/maximum dates and if the user must enter a date range:

Restricted dates and date range picker in submit view:

Results in summary / responses view:


Signed-off-by: Christian Hartmann chris-hartmann@gmx.de