-
Notifications
You must be signed in to change notification settings - Fork 0
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
Redo OTM modeling #5
Conversation
src/dil_demo/master_data.clj
Outdated
(map #(vector (get-in config [% :eori]) | ||
(get-in config [% :site-name]))) | ||
(into {}))] | ||
(fn carriers-wrapper [req] |
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.
master-data-wrapper
src/dil_demo/otm.clj
Outdated
(defn check! [type value] | ||
(when-let [spec ({:consignments ::consignment | ||
:trips ::trip | ||
:transport-orders ::transport-order} |
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.
dus :consignments
is 1 consignment? waarom gebruiken we niet de spec keys?
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.
Alle data wordt in een enkele map opgeslagen en alle :consignments
dus onder die key. Ik verhuis dit naar de store
namespace voor de duidelijkheid.
:flash flash | ||
:title title | ||
:site-name site-name)) | ||
params-> (fn params-> [params] |
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.
Kan je ergens afdwingen dat deze altijd aangeroepen wordt? Kan dit als middleware?
Als deze select-keys een goed idee is, moet dit dan niet ook bij tms.web en wms.web?
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.
In de andere namespaces worden nooit de params zomaar als object geaccepteerd vandaar dat het in die andere namespaces niet nodig is. Voor middleware vind ik het veel te speciek en kan het in de weg zitten als er andere schermen bij komen die geen crud operaties doen maar ander soortige formulieren hebben.
No description provided.