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

[website] onboarding next iteration #30986

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@kea14
Copy link
Contributor

kea14 commented Feb 11, 2019

PR creation

@robodoo robodoo added the seen 🙂 label Feb 11, 2019

@kea14 kea14 requested a review from qsm-odoo Feb 11, 2019

@C3POdoo C3POdoo added the RD label Feb 11, 2019

@robodoo robodoo added the CI 🤖 label Feb 11, 2019

@kea14 kea14 force-pushed the odoo-dev:master-onboarding-next-iteration-aul branch from ba9284a to 80b3386 Feb 11, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 11, 2019

@kea14 kea14 force-pushed the odoo-dev:master-onboarding-next-iteration-aul branch from 80b3386 to e610365 Feb 12, 2019

@robodoo robodoo removed the CI 🤖 label Feb 12, 2019

@qsm-odoo
Copy link
Contributor

qsm-odoo left a comment

@kea14 Sorry but as always with those magnificent specs, you will have to undo stuff. Also, could you split your work in multiple commits when those are unrelated stuff ? Thank you.

{
text: _t("Cancel"),
close: true,
}

This comment has been minimized.

@qsm-odoo

qsm-odoo Feb 12, 2019

Contributor

There already was buttons before, so this is obviously not the right way ?

This comment has been minimized.

@kea14

kea14 Feb 13, 2019

Author Contributor

Indeed this is useless as this is not the prob and I should have removed that, sorry 😄

This comment has been minimized.

@kea14

kea14 Feb 13, 2019

Author Contributor

Concerning buttons translation, I'm not sure on how to fix it, do you have an idea ?

Show resolved Hide resolved addons/website/static/src/js/menu/content.js Outdated
Show resolved Hide resolved addons/website/views/snippets.xml
Show resolved Hide resolved addons/website/views/snippets.xml
Show resolved Hide resolved addons/website/views/website_templates.xml

@robodoo robodoo added the CI 🤖 label Feb 12, 2019

@kea14 kea14 force-pushed the odoo-dev:master-onboarding-next-iteration-aul branch from e610365 to d4c3659 Feb 13, 2019

@robodoo robodoo removed the CI 🤖 label Feb 13, 2019

kea14 added some commits Feb 13, 2019

[IMP] website,web_editor: usability issues
Usability issues concerning website

task-id: 1939869
[IMP] website: usability issues
Usability issues concerning editor usability

task-id: 1939869

@kea14 kea14 force-pushed the odoo-dev:master-onboarding-next-iteration-aul branch from d4c3659 to e2eb54b Feb 13, 2019

@robodoo robodoo added the CI 🤖 label Feb 13, 2019

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