search

metosin / sieppari

Small, fast, and complete interceptor library for Clojure/Script
Eclipse Public License 2.0
207 stars 21 forks source link

Fix protocol extension (for cljs) #35

Closed zelark closed 4 years ago

zelark commented 4 years ago

Relates to #34

https://stackoverflow.com/questions/16894465/extend-protocol-in-clojurescript-to-get-value-from-native-js-object

ikitommi commented 4 years ago

Hi. Actually default would cover all cases: https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/core.cljc#L1312-L1328. Could you change to that?

zelark commented 4 years ago

Hi, sure.

zelark commented 4 years ago

Just changed. Didn't know about default. Thank you.

ikitommi commented 4 years ago

Thanks!