changes in init url

group url set with slash and without slash also.
if browse by topic and resources urls is not working,
shift this two url above


On Sat, Jun 7, 2014 at 11:13 AM, sunnychaudhari <notifications@github.com>
wrote:

> In gnowsys-ndf/gnowsys_ndf/ndf/urls/__init__.py:
>
> > @@ -38,9 +38,9 @@
> >      (r'^(?P<group_id>[^/]+)/ajax/', include('gnowsys_ndf.ndf.urls.ajax-urls')),
> >      (r'^(?P<group_id>[^/]+)/', include('gnowsys_ndf.ndf.urls.user')),
> >     # (r'^(?P<group_id>[^/]+)/',include('gnowsys_ndf.ndf.urls.group')),
> > -    url(r'^(?P<group_id>[^/]+)$','gnowsys_ndf.ndf.views.group.group_dashboard', name='groupchange'),
> > -    (r'^(?P<group_id>[^/]+)/browse topic', include('gnowsys_ndf.ndf.urls.browse_topic')),
> > -    (r'^(?P<group_id>[^/]+)/browse resource', include('gnowsys_ndf.ndf.urls.browse_resource')),
> > +    url(r'^(?P<group_id>[^/]+)/$','gnowsys_ndf.ndf.views.group.group_dashboard', name='groupchange'),
> > +    (r'^(?P<group_id>[^/]+)/browse topic/', include('gnowsys_ndf.ndf.urls.browse_topic')),
> > +    (r'^(?P<group_id>[^/]+)/browse resource/', include('gnowsys_ndf.ndf.urls.browse_resource')),
>
> Dhiru, i think you made a change in urls, because groupchange url which
> you wanted to work ,was not going in to that context, may be it was
> reverting into browse topic or browse resource context. So you put that
> "slash" and that url gets worked. right? , or whatever might be the reason.
> Thats not an issue, you did a good job by allowing "/" in group url , but
> due to that slash("/") appended in line no 42 & 43, browse topic and browse
> resources apps are not working now. Lets make it generic, why only group,
> we require all urls to be worked with "/" or without "/".
>
> Lets make a common solution for all urls, as GN reported a bug regarding
> urls, that "if you put the slash("/") at the end of url it gives an error",
> so can we make something like for any url if it has not slash at the end ,
> urls automatically take slash("/") appended at the end and goes into its
> appropriate app context"
>
> For that we have to define all urls unique from each other, we can do that
> by putting some extra contents in urls which uniquely identifies context.
> or we can use some other django's conventional patterns to define urls
> uniquely which allows slash"/" appended at the end of urls.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gnowledge/gstudio/pull/508/files#r13518098>.
>



-- 
Thanks and regards,
Dhirendra singh
Project Assistant
Gnowledge lab
HBCSE,TIFR
Mumbai.