Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.0][MIG] website_lazy_load_image #656

Conversation

hugho-ad
Copy link

No description provided.

tarteo and others added 11 commits July 30, 2019 20:18
[FIX] Lint

[IMP] img tranformation in ir.ui.view

[FIX] jsdocs and flake8

[ADD] Excluding images and more documentation

[IMP] Better usage text

[FIX] Flake and jslint

[ADD] Tests

[FIX] missing EOF new line

[IMP] Use Savepointcase for tests

[IMP] Use Savepointcase for tests
html.fromstring function.

Before this commit, the render_template has an issue with views retrieved
from a controller, like the wizard of event detail registration, the
content was wrapped into an html DOM, this leads a wrong modal
behavior.

FIX OCA#551
@pedrobaeza pedrobaeza added this to the 12.0 milestone Jul 30, 2019
Copy link

@oscarolar oscarolar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@hugho-ad
Copy link
Author

hugho-ad commented Aug 3, 2019

@simahawk
could you review? please

@hugho-ad
Copy link
Author

hugho-ad commented Aug 5, 2019

@yajo could you review please

Copy link

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just code review. LG

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza
Copy link
Member

@yajo is on holidays, so go ahead if you think so.

@simahawk
Copy link

simahawk commented Aug 6, 2019

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Rebased to 12.0-ocabot-merge-pr-656-by-simahawk-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 6, 2019
Signed-off-by simahawk
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a827d1f. Thanks a lot for contributing to OCA. ❤️

PS: Don't worry if GitHub says there are unmerged commits: it is due to a rebase before merge. All commits of this PR have been merged into 12.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants