Skip to content
Snippets Groups Projects
Commit a416ca66 authored by Xavier Morel's avatar Xavier Morel
Browse files

[REM] remove fixed/updated todos

parent fdc63e3a
No related branches found
No related tags found
No related merge requests found
...@@ -133,10 +133,6 @@ Let's move our 2 pseudo-templates from inline strings to actual templates: ...@@ -133,10 +133,6 @@ Let's move our 2 pseudo-templates from inline strings to actual templates:
.. patch:: .. patch::
.. todo:: how can I access a QWeb template from a auth=none controller?
explicitly fetch a registry using request.session.db? That's a bit
horrendous now innit?
This simplifies the controller code by moving data formatting out of it, and This simplifies the controller code by moving data formatting out of it, and
generally makes it simpler for designers to edit the markup. generally makes it simpler for designers to edit the markup.
...@@ -279,13 +275,6 @@ To fix the issue, we can simply add ``website`` as a dependency to ``academy`` ...@@ -279,13 +275,6 @@ To fix the issue, we can simply add ``website`` as a dependency to ``academy``
.. patch:: .. patch::
.. todo:: website dispatch overrides blows up on auth=none (implicitly
inherits website's index -> ``website_enabled`` -> tries to access
``request.registry['website']`` even though ``request.registry is
None`` because ``auth='none'``)
also template issues (see above) (enabled website to "fix")
This will cause ``academy``'s index page to overwrite ``website``'s. This will cause ``academy``'s index page to overwrite ``website``'s.
Reload `your openerp`_. Your old index page is back. Reload `your openerp`_. Your old index page is back.
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment