feat: impl datetime dropdown #74
Conversation
samsoft00
commented
May 21, 2023
•
samsoft00
commented
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
model/date/core/package.json
Outdated
| "date-fns": "^2.29.3" | ||
| "date-fns": "^2.29.3", | ||
| "dayjs": "~1.11.7", | ||
| "moment": "^2.29.4" |
There was a problem hiding this comment.
Sam, do we really need all three date packages?
There was a problem hiding this comment.
Updated, remove date-fn and moment
| { key: "DD/MM/YY", value: "DD/MM/YY (22/01/06)" }, | ||
| { key: "X", value: "X (1137971045)" } | ||
| ] | ||
|
|
There was a problem hiding this comment.
Great that we are making a drop down. But is this too complicated? Check what Airtable does . 5 or 6 simple options, with a live example so you can see what it looks like.
Also, the current date in cozemble does not support time. My thinking so far was to model time as its own type....but not sure if that is a good idea.
There was a problem hiding this comment.
Let's discuss this, I'm willing to make adjustments.
|
|
||
| <label class="label" for="date-format-selector">Date format selector</label> |
There was a problem hiding this comment.
You're getting better with svelte :)
| </div> | ||
| <!-- | ||
| <label class="label" for="date-format-input">Date format:</label> |
There was a problem hiding this comment.
Delete this commented out code
There was a problem hiding this comment.
Sure, I'll handle that.
Metehan-Altuntekin
force-pushed
the
main
branch
from
3e76d9d
to
3bf2206
Compare
