-
Notifications
You must be signed in to change notification settings - Fork 23.5k
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
[IMP] website: improve configurator logo upload #145047
[IMP] website: improve configurator logo upload #145047
Conversation
ab06443
to
06830df
Compare
7ec43f3
to
154f9ba
Compare
6c67c5e
to
31a06d9
Compare
@robodoo delegate=sobo-odoo |
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.
Here is my review 😉
It seems to work well except for a detail (see "behavior" comment)
I also have a few questions so the changes will depend on your answers 🙂
addons/website/static/src/client_actions/configurator/configurator.js
Outdated
Show resolved
Hide resolved
addons/website/static/src/client_actions/configurator/configurator.js
Outdated
Show resolved
Hide resolved
addons/website/static/src/client_actions/configurator/configurator.js
Outdated
Show resolved
Hide resolved
addons/website/static/src/client_actions/configurator/configurator.js
Outdated
Show resolved
Hide resolved
f183862
to
c436795
Compare
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.
Last nitpicks and then I will r+ 🙂
addons/website/static/src/client_actions/configurator/configurator.js
Outdated
Show resolved
Hide resolved
Before this commit, when a user was uploading a logo for a website via the website configurator, the logo was added as an attachment but the potential previous logo was not deleted. This commit fixes this by deleting the previous logo when a new one is uploaded. Steps to reproduce the issue: - Create a new website - Fill the configurator until you can upload a logo - Upload a logo, then upload another one - Finish the configurator - Edit a page of the website - Change an image => You can see the first logo you uploaded in the list of images. task-3609329
This commit adds a button to remove the website logo in the website configurator. task-3609329
This commit permits to automatically select the colors of the logo after uploading it. task-3609329
Before this commit, when a user uploaded a webp logo through the website configurator, the colors of the logo were not detected. This commit adds the possibility to detect the colors of a webp logo. task-3609329
c436795
to
cd0f77c
Compare
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.
Alright, let's r+ ! 😉
@robodoo rebase-ff r+
Merge method set to rebase and fast-forward. |
Before this commit, when a user was uploading a logo for a website via the website configurator, the logo was added as an attachment but the potential previous logo was not deleted. This commit fixes this by deleting the previous logo when a new one is uploaded. Steps to reproduce the issue: - Create a new website - Fill the configurator until you can upload a logo - Upload a logo, then upload another one - Finish the configurator - Edit a page of the website - Change an image => You can see the first logo you uploaded in the list of images. task-3609329 Part-of: #145047
This commit adds a button to remove the website logo in the website configurator. task-3609329 Part-of: #145047
This commit permits to automatically select the colors of the logo after uploading it. task-3609329 Part-of: #145047
This PR improves the logo detection part of the website configurator.
Summary of changes:
task-3609329