Skip to content

[10.0] web_environment_ribbon forward port#661

Merged
moylop260 merged 3 commits intoOCA:10.0from
acsone:10.0-ribbon-vars-sbi
Jul 13, 2017
Merged

[10.0] web_environment_ribbon forward port#661
moylop260 merged 3 commits intoOCA:10.0from
acsone:10.0-ribbon-vars-sbi

Conversation

@sbidoul
Copy link
Copy Markdown
Member

@sbidoul sbidoul commented Jul 2, 2017

Forward port of #621 with a couple of small improvements:

  • make the backend model abstract as it does not need a db table
  • make the backend model easier to extend with additional variables

ThomasBinsfeld and others added 3 commits July 2, 2017 21:49
…(eg db_name) (OCA#621)

* [ADD] .eggs in gitignore

* [IMP] Web Environment Ribbon: add backend
This commit adds a simple bakend to facilitate inheritance

* [IMP] Web Environment Ribbon: add the databse name
This commit adds the database name under the ribbon name, in the ribbon

* [IMP] Web Environment Ribbon: prepare method for ribbon name

* [IMP] README: explain how to use db_name
@yajo yajo added this to the 10.0 milestone Jul 3, 2017
@lfreeke
Copy link
Copy Markdown

lfreeke commented Jul 13, 2017

👍 (test)

Copy link
Copy Markdown

@elicoidal elicoidal left a comment

Choose a reason for hiding this comment

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

👍

@moylop260
Copy link
Copy Markdown

moylop260 commented Jul 13, 2017

The color changed for this version:

  • screen shot 2017-07-13 at 10 06 04

Shouldn't be same color like as old version?

  • screen shot 2017-07-13 at 10 06 58

@sbidoul
Copy link
Copy Markdown
Member Author

sbidoul commented Jul 13, 2017

@moylop260 on runbot, the system parameter has been changed:
image

I assume another tester changed it manually, because this PR does not touch de default value.

@moylop260 moylop260 merged commit f2f4ac1 into OCA:10.0 Jul 13, 2017
@moylop260
Copy link
Copy Markdown

Thanks for clarify and migration!

@sbidoul sbidoul deleted the 10.0-ribbon-vars-sbi branch July 13, 2017 15:38
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.

6 participants