search

eudoxia0 / lucerne

A web framework for Common Lisp, built on Clack
http://borretti.me/lucerne/
142 stars 19 forks source link

:get method doesn't work #6

Closed ghost closed 9 years ago

ghost commented 9 years ago

When I try this:

@route app "/index" (defview hello () (with-params (test) (respond (format nil "Hello, world! ~a" test))))

This doesn't work. Should print "Hello, world! lisp" when the url is "/index?test=lisp" ?

eudoxia0 commented 9 years ago

Lucerne uses myway for routing, so you need to make that URL "/index*". This is, I'm pretty sure, a bug in myway.

eudoxia0 commented 9 years ago

After raising the issue on the myway repo and hearing that it's out-of-scope, I think I'll change the router to fix this.

ghost commented 9 years ago

'/index' is acceptable for me, though a little bit of strange. After this "problem" is fixed, will '/index' still work?

ghost commented 9 years ago

Previous comment has just been updated. I didn't notice Markdown syntax is supported.

eudoxia0 commented 9 years ago

That is a good question. I'm not sure it would, I suppose it would depend on just how it was implemented. I'll have to write it first and see.

ghost commented 9 years ago

Oh OK, thanks for your work on this anyway :)