[Toaster] [review-request] adamian/20150507_toasterui_fixes

Michael Wood michael.g.wood at intel.com
Fri May 8 09:53:22 PDT 2015


Spotted one error I think

The project page link  r'^project/(?P<pid>\d+)$' should be 
r'^project/(?P<pid>\d+)/$

i.e.

-        url(r'^project/(?P<pid>\d+)$', 'project', name='project'),
+        url(r'^project/(?P<pid>\d+)/$', 'project', name='project'),

Your django might not have reloaded the urls so didn't catch it maybe?

recreate:

1. go to /projects/
2. Select a project - 404

Michael

On 08/05/15 17:39, Michael Wood wrote:
> Looks fine to me.
>
> Michael
>
> On 07/05/15 18:13, Damian, Alexandru wrote:
>> Hello,
>>
>> This a refactoring of the URL structure to be more REST-ful.
>> It remove the usage of the server-side user session to remember 
>> currently selected project, making navigation more robust, opening 
>> API for usage to other tools, and passing the crawl test.
>>
>> Cheers,
>> Alex
>>
>> -- 
>> Alex Damian
>> Yocto Project
>> SSG / OTC
>>
>>
>



More information about the toaster mailing list