[18.0][IMP] add opt-in setting for automatic PDF invoice preview opening - #3666
vancouver29 wants to merge 1 commit into
Conversation
pedrobaeza
left a comment
There was a problem hiding this comment.
I think the option should be set per user, not per system.
| "license": "LGPL-3", | ||
| "installable": True, | ||
| "depends": ["web", "web_tour", "mail"], | ||
| "depends": ["web", "web_tour", "mail", "account"], |
There was a problem hiding this comment.
This change is a total blocker. You can't depend on account.
I was agnostic about this and could see advantage for both, but lets go with your suggestion. |
24f4964 to
2a87da8
Compare
|
@vancouver29 I cant see this setting in the users preferences Also, the screenshot in the readme is not working. Best |
Hi Jan @smaddlsoft , you should change that option in settings module, not in profile references. Here is the click path: Settings (Module/App) -> Users & Companies -> Users -> choose the User -> Preferences -> check/uncheck the checkbox in the File Viewer. best regards |
smaddlsoft
left a comment
There was a problem hiding this comment.
kindly move the setting to the user preferences accessible by each user, not via settings->users, which most users wont be able to access.
| ## Auto-open invoice preview | ||
|
|
||
| Enable *Auto-open Invoice Preview* in your user preferences | ||
| (*Your Profile → Preferences*). When enabled, opening an invoice |
There was a problem hiding this comment.
currently its not in the users preferences which are accessible directly by the user, but only in the settings->users, which obviously is not accesible by each user.
Kindly move to the user-accsessible user preference window, where also the chatter position is maintained, which is a logical place to put also this opt-in, because both are somewhat related.
…ice preview opening
2a87da8 to
c9719b4
Compare
Improve the PDF preview in
web_responsiveModuleImprovements #3334
See this comment for context.