it-projects-llc / odoo-saas-tools

Odoo SaaS Tools — tools for SaaS Businesses. Sale and manage Odoo databases.
https://saas.it-projects.info
GNU Lesser General Public License v3.0
543 stars 578 forks source link

[11.0] [FIX] 'website' object has no attribute 'render' #758

Closed serpentcs-dev1 closed 6 years ago

serpentcs-dev1 commented 6 years ago

@yelizariev The render method is removed from the website, it is deprecated. Now directly request,render is used!

Please find the PR which explains that change

https://github.com/odoo/odoo/commit/8a399930f316fc4d65828441d86301d4a8dfa669

serpentcs-dev1 commented 6 years ago

To reproduce, the error is generated when redirecting to this template http://localhost:8069/saas_portal_templates/select-template

internal server error

serpentcs-dev1 commented 6 years ago

This will resolve #408 and #654

@njeudy @zimbora69 @GoodERPJeff

njeudy commented 6 years ago

@yelizariev can we merge #746 because all this kind of error are fixed in V11. @darshan-serpent have you tested with this PR #746 ?

serpentcs-dev1 commented 6 years ago

@njeudy Yes right, I have tested it in v11, it works!

Also I have attached the odoo v11 commit which helps to understand the change : https://github.com/odoo/odoo/commit/8a399930f316fc4d65828441d86301d4a8dfa669

serpentcs-dev1 commented 6 years ago

can we merge #746 because all this kind of error are fixed in V11.

@njeudy Hello Nicolas, actually I tried to locate that change in below branch and your PR too, but could not find : https://github.com/njeudy/odoo-saas-tools/blob/11.0-fixes-run01/saas_portal_templates/controllers/main.py#L15

IMHO this PR has the minor change to merge

Also I am waiting for 746 to get merge, as it is resolving issues, and many other code and styling improvements :+1:

cc @yelizariev

yelizariev commented 6 years ago

746 is merged

Sorry for late reply. I close this PR for a while