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.

Sign up for GitHub

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

Jump to bottom

Instance-level option to disable account detail expansion #78

Merged
merged 2 commits into from
Apr 16, 2018
Merged

Instance-level option to disable account detail expansion #78

merged 2 commits into from
Apr 16, 2018

Conversation

sbidoul
Copy link
Member

@sbidoul sbidoul commented Apr 14, 2018

Closes #74

@sbidoul
Copy link
Member Author

sbidoul commented Apr 14, 2018

@jbeficent

This was referenced Apr 14, 2018
Closed
Closed
@JordiBForgeFlow JordiBForgeFlow mentioned this pull request Apr 16, 2018
Merged
JordiBForgeFlow added a commit to ForgeFlow/l10n-spain that referenced this pull request Apr 16, 2018
@JordiBForgeFlow
a quitar: dependencia con OCA/mis-builder#78
cc08b4e
JordiBForgeFlow added a commit to ForgeFlow/l10n-spain that referenced this pull request Apr 16, 2018
@JordiBForgeFlow
quitar: dependencia con MIS Builder OCA/mis-builder#78
dd56ced
JordiBForgeFlow
JordiBForgeFlow approved these changes Apr 16, 2018
View reviewed changes
Copy link
Member

@JordiBForgeFlow JordiBForgeFlow left a comment

Choose a reason for hiding this comment

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

👍 Tested in OCA/l10n-spain#826

@sbidoul
Copy link
Member Author

sbidoul commented Apr 16, 2018 •
edited
Loading

@benwillig @gva-acsone this may be of interest to you

pedrobaeza
pedrobaeza approved these changes Apr 16, 2018
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Working flawlessly! Thanks, Stéphane.

About the new page, do you think it's a good idea to have it for 2 options?

@sbidoul
Copy link
Member Author

sbidoul commented Apr 16, 2018

@pedrobaeza there will be more in that page in the future (eg from #42 and #82)

@sbidoul sbidoul merged commit 901aaa9 into OCA:10.0 Apr 16, 2018
@sbidoul sbidoul deleted the 10.0-no_auto_expand_accounts-sbi branch April 16, 2018 19:32
@pedrobaeza
Copy link
Member

OK, I see.

@JordiBForgeFlow JordiBForgeFlow mentioned this pull request Apr 17, 2018
Closed
JordiBForgeFlow added a commit to ForgeFlow/l10n-spain that referenced this pull request Apr 20, 2018
@JordiBForgeFlow
quitar: dependencia con MIS Builder OCA/mis-builder#78
09672fc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JordiBForgeFlow JordiBForgeFlow JordiBForgeFlow approved these changes

@pedrobaeza pedrobaeza pedrobaeza approved these changes

Assignees
No one assigned
Labels
10.0 needs review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sbidoul @pedrobaeza @JordiBForgeFlow