metosin / metosin/schema-tools
Support for joda-time
Open
Nobody has claimed this yet.
enhancement
help wanted
- Dominant language
- Clojure
- Stars
- 110
- Forks
- 15
- Avg merge
- 6m
- Merged PRs (30d)
- 1
Description
For library ergonomics, would be great if there was bindings for joda-time classeses too. could be conditional, e.g. if the classes are found in classpath. Many libs (like https://github.com/metosin/metosin-common) still use joda...
(defmethod transform-type DateTime [_ _] {:type "string" :format "date-time"})
(defmethod transform-type LocalDate [_ _] {:type "string" :format "date"})
(defmethod transform-type LocalTime [_ _] {:type "string" :format "time"})
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the transform-type multimethod and the existing handling for date/time classes. Add conditional support for the listed joda-time DateTime, LocalDate, and LocalTime classes, then verify that their schema types and formats match the examples in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- clojure
- Domain
- api
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100