I'm looking to integrate Dialog API into XML sitemap (see #661780: AJAX-ify the type settings on admin/settings/xmlsitemap). A couple things are tricking me up:
1. You can't use links like 'foo/bar/nojs'. There has to be a trailing slash at the end like 'foo/bar/nojs/'. Took me like 15 minutes to figure out why it wasn't working right. :)
2. The 6.x-1.x and 7.x-1.x versions seem out of sync and I was wondering if you'd be willing to keep development synchronized for both branches. I know a lot of the module's requirements made it into D7, but there are things like dialog_get_form() and extras in the 6.x-1.x version. Since lots of modules are starting to be ported to D7, it would be great to not have confusion of why these two versions have different features.
Comments
Comment #1
zroger commentedpoint #1 was fixed in ctools #565808: nojs/ajax replacement back in september. are you positive that you are using the latest ctools release?
As for #2, I'm starting to work on that now. RobLoach initially started the 7.x branch and I haven't made a very good effort to keep it up to date. Some things will be much harder than others, for instance all of the form handling stuff which depends on
ctools_build_form.Comment #2
dave reid#1 was on Drupal 7, so no, didn't have anything to do with CTools. Should I file a Drupal core issue?
Comment #3
dave reidI moved #565808: nojs/ajax replacement to up-port to D7 core, so if you could review, that'd be great. :)
Comment #4
dave reidAs my answer was provided, I'm marking this as fixed. Keep up the great work Roger! You'll surely have more users with Dialog API since I just officially added integration with XML sitemap and it'll be available in the upcoming alpha release of the 2.x rewrite. :)
Comment #5
robloachGet out that pimp-stick, son!
There's so much in Drupal core now that's not needed Drupal 7 branch that was in the Drupal 6 branch. The problem is filtering through the core bugs and pushing in features that would be helpful to us. Some of what this module implements is core worthy, and we should figure out what that is.
Comment #6
BenK commentedSubscribing.....
Comment #7
sun@Rob: Can we compile a list? :)
Comment #8
robloachHmmmmm....
Comment #9
andypostsubscribe
Comment #10
podaroksubscribe
Comment #11
andypostWhat's left here for stable release
Comment #12
mustanggb commentedDependencies from #8 are fixed.
I think the included sub-modules and examples should work before rolling a release though: #1457230: Login or Register does not validate. 7.x-1.x-dev
Comment #13
devin carlson commentedThe 7.x-1.x branch of the Dialog module is unsupported.