-
Notifications
You must be signed in to change notification settings - Fork 279
[FIX] theme_test_custo: correct tour trigger to look for svg images #1102
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
Conversation
@robinlej @loco-odoo What do you think ? Maybe the Img component should have an attribute with the source to be able to check it's value ? <svg t-att-data-src="props.src" ... /> |
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641
e2130ab
to
dd45166
Compare
I rebased an turns out another change has been made that breaks the tour odoo/odoo@eca4d1b#diff-d3ccab48ea391b9812d735e007298e1d86e6b93b8848fb8204b5be6a6524c40a |
@robodoo delegate=robinlej |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Should I r+ @hubvd ? Can you confirm that it's ready?
Yes, thanks :) |
@robodoo r+ |
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
Since [1], the test "test_01_menu_hierarchies" fails. Note that the check is not as strong as before as we cannot check the source of the image. Since [2], the btn-primary is now btn-secondary --- [1]: odoo/odoo@3cc5ff6 [2]: odoo/odoo@eca4d1b runbot-226641 closes #1102 Signed-off-by: Robin Lejeune (role) <[email protected]>
@hubvd can you create the backport in 18.4? |
Sure |
Since 1, the test "test_01_menu_hierarchies" fails.
Note that the check is not as strong as before as we cannot check the
source of the image.
Since 2, the btn-primary is now btn-secondary
runbot-226641