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

Migrate web_easy_switch_company to the last v8 #18

Merged
merged 5 commits into from
Jul 26, 2014
Merged

Migrate web_easy_switch_company to the last v8 #18

merged 5 commits into from
Jul 26, 2014

Conversation

AdilHoumadi
Copy link

  • change route in controller
  • adapt the JS to call the backend
  • modify the css to graft the widget in the toolbar

- change route in controller
- adapt the JS to call the backend
- modify the css to graft the widget in the toolbar
@hbrunn
Copy link
Member

hbrunn commented Jul 17, 2014

please fix the minor flake8 errors as indicated above
[edit] they were above my input field, but you'll see them under my comment now

Adil Houmadi added 2 commits July 17, 2014 22:41
./web_easy_switch_company/controllers/main.py:26:1: E302 expected 2 blank lines, found 1
./web_easy_switch_company/controllers/main.py:32:9: F841 local variable 'context' is assigned to but never used
./web_easy_switch_company/controllers/main.py:36:1: W391 blank line at end of file
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 81f5e93 on taktik:master into 0cc75aa on OCA:master.

@hbrunn
Copy link
Member

hbrunn commented Jul 21, 2014

Thank! Two last requests: Please remove the module from unported and also remove placeholder, which only served to make travis run (it chokes on empty addons paths)

- The first module is here so we need to remove the placeholder folder
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 6856ea4 on taktik:master into 0cc75aa on OCA:master.

@hbrunn
Copy link
Member

hbrunn commented Jul 22, 2014

👍

@pedrobaeza
Copy link
Member

You have to change icon path from /static/src/img to /static/description in order to get displayed.

@pedrobaeza
Copy link
Member

The rest of the module LGTM. We still have logo from other companies issue, but I can't imagine any solution. I will push my thumb up when you do this change.

Regards.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 13faa40 on taktik:master into 0cc75aa on OCA:master.

@pedrobaeza
Copy link
Member

👍

Thanks

pedrobaeza added a commit that referenced this pull request Jul 26, 2014
[MIG] web_easy_switch_company
@pedrobaeza pedrobaeza merged commit 529e7cb into OCA:master Jul 26, 2014
@rvalyi
Copy link
Member

rvalyi commented Jul 26, 2014

👍

leemannd pushed a commit to camptocamp/web that referenced this pull request May 29, 2018
Remove VERSION for the .dockerignore file
vrenaville pushed a commit to camptocamp/web that referenced this pull request Jul 19, 2018
Use camptocamp/odoo-project:10.0-2.1.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants