-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add FOSJsRoutingBundle #1765
Add FOSJsRoutingBundle #1765
Conversation
Please check if you see the route in the RoutesModule admin interface. If not, click reload routes and check again. |
@@ -40,6 +40,7 @@ | |||
"drak/doctrine1": "dev-master", | |||
"drak/smarty": "dev-master", | |||
"elao/web-profiler-extra-bundle" : "dev-master", | |||
"friendsofsymfony/jsrouting-bundle": "@stable", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you want stable, use a version number/range.
@cmfcmf - I think I will close this PR unless it can be of a help to you somehow. I thought it would be relatively little simple thing, but (as usual) I was wrong and in over my head. How would you like me to proceed? |
I was preferring not to use the command line, but was trying to see if anything was working via that method. let me rephrase my previous comment: I don't want to be working on this anymore 😉 would you please take it over ❓ what is the best way to hand off the work to you? just close the PR or something else? |
@craigh Ok I understand. So.. Please remove your last commit and merge afterwards. It's working. |
remove the commit ca47292 ? |
Yes. This is not needed (works without the locale parameter). |
commit is removed. merge at your whim 🐐 |
Implementation example: zikula/ExtensionLibrary@a6226e6...7c16e80 |
refs #1552
ping @cmfcmf @Guite
I am pretty sure this is correct, but I have yet to figure out to make it work in my own javascript file. I know that it is trying to work, but failing.
for example
var route = Routing.generate('zikulaextensionlibrarymodule_ajax_setversionstatus', null, true);
returns:
Error: The route "zikulaextensionlibrarymodule_ajax_setversionstatus" does not exist.
so, I need some ideas.