Skip to content
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

FlowRouter Support #343

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

FlowRouter Support #343

wants to merge 8 commits into from

Conversation

AlexeyMK
Copy link
Collaborator

@AlexeyMK AlexeyMK commented Nov 5, 2015

[Still WIP]

@gterrono, looks like MDG is leaning towards just supporting FlowRouter. For Houston to work with FlowRouter is a non-trivial rewrite, since we rely on template data context and router level subscriptions pretty heavily and FlowRouter doesn't particularly like that approach.

That said, an (as you can see) temporary hack includes just passing data through anyway.

A bunch of stuff is still broken (had a few hours during a hackday to poke at this) but figured I'd share and see if you or others had time to contribute.

@adrianmcli
Copy link

adrianmcli commented Dec 18, 2015

How well does this work? I'm still on the lookout for a no-config admin solution.

@rafa-munoz
Copy link

rafa-munoz commented Feb 15, 2016

The intention of this PR is good, but unfortunately I think it was not tested.

When I go to /admin and introduce the login data, then I get a blank page and no visible errors on console. The same all the time for every click that I do in any link: white page and only reloading I can access the page. Any clue @AlexeyMK ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants