search

mick / billboard

Control content in a browser via a simple HTTP API. Great for office wall mounted displays
MIT License
12 stars 6 forks source link

Fix problem resulting from view URL change #30

Open daguar opened 11 years ago

daguar commented 11 years ago

(@mick, you mentioned this last night, but I haven't investigated it at all)

Just a note, in case it's helpful, that you can edit my diff to retrieve the route input in the same way you do the ending param:

/:first_path_text(screen[s]?)/:name

:first_path_test will be available in the same way as :name, but will carry only the value screen or screens

Let me know when you have a chance what the issue is exactly and I'll try to patch it up.

mick commented 11 years ago

Its in the frontend JS. I very naively accept anything after 8 characters as the screen name see main.js so instead of "main" you'll get "/main" if url contains the extra "s".

This should be changed to a regex that matches the same one you are using for routing in app.js