luciano/reitit
1
0
Fork 0
mirror of https://github.com/metosin/reitit.git synced 2025-12-24 02:48:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix event definitions for routes with parameters

Browse source 
The current implementation will fail for `(re-frame/dispatch [::navigate ::some-route {:some :params}])`.

Update the definitions of `::navigate` and `::navigated!` events to accept the full route arguments.
This commit is contained in:
dimovich 2019-09-08 18:28:50 +03:00 committed by Tommi Reiman
parent ca7403ccab
commit 031a777c8c
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
examples/frontend-re-frame/src/cljs/frontend_re_frame/core.cljs
View file

@ -18,7 +18,7 @@
(re-frame/reg-event-fx
::navigate
(fn [db [_ route]]
(fn [db [_ & route]]
;; See `navigate` effect in routes.cljs
{::navigate! route}))
@ -59,8 +59,8 @@
;; Triggering navigation from events.
(re-frame/reg-fx
::navigate!
(fn [k params query]
(rfe/push-state k params query)))
(fn [route]
(apply rfe/push-state route)))
;;; Routes ;;;