Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

Commit

Permalink
publish muk_web_utils - 12.0
Browse files Browse the repository at this point in the history
  • Loading branch information
keshrath committed Mar 11, 2019
1 parent 778159b commit 63de0ee
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion muk_web_utils/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
{
"name": "MuK Web Utils",
"summary": """Utility Features""",
"version": "12.0.2.8.21",
"version": "12.0.2.8.22",
"category": "Extra Tools",
"license": "AGPL-3",
"author": "MuK IT",
Expand Down
2 changes: 1 addition & 1 deletion muk_web_utils/static/src/js/fields/binary.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ fields.FieldBinaryFile.include({
}).done(function(result) {
this.max_upload_size = result.max_upload_size * 1024 * 1024;
}.bind(this));
return $.when(this._super.apply(this, arguments), def);
return this._super.apply(this, arguments);
},
_renderReadonly: function () {
this._super.apply(this, arguments);
Expand Down
2 changes: 1 addition & 1 deletion muk_web_utils/static/src/js/fields/image.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ fields.FieldBinaryImage.include({
}).done(function(result) {
this.max_upload_size = result.max_upload_size * 1024 * 1024;
}.bind(this));
return $.when(this._super.apply(this, arguments), def);
return this._super.apply(this, arguments);
},
_render: function () {
this._super.apply(this, arguments);
Expand Down

8 comments on commit 63de0ee

@luisvzqz
Copy link

Choose a reason for hiding this comment

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

Thank you very much does this lines are included in the v12 branch ??? -LV

@keshrath
Copy link
Member Author

Choose a reason for hiding this comment

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

Well yes it's a commit of the 12.0 branch. Why do you ask?

@luisvzqz
Copy link

Choose a reason for hiding this comment

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

Because I just download it a week ago, so I just downloaded again today and I am installing it, thanks, -LV

@luisvzqz
Copy link

Choose a reason for hiding this comment

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

The bug was corrected and works fine !!! the only problem that I have is that the button Settings doesn't open anything, if you browse in the search menu it works great !!!
Question: How come the button Settings doesn't work ?

@keshrath
Copy link
Member Author

Choose a reason for hiding this comment

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

Can you take a look in the browser console? any errors?

@spacimek
Copy link

Choose a reason for hiding this comment

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

For me all works well - I could not get to General Settings due to issues with settingView, but this has fixed it. Thank you very much!

@luisvzqz
Copy link

Choose a reason for hiding this comment

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

No, it doesn't show anything, maybe it is the settings / dashboard ? that I have it wrong ?? because it doesn't do anything ?? thanks for the advise !!

@luisvzqz
Copy link

Choose a reason for hiding this comment

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

Now it works good the Button of Settings !! it seems to be something related with the cookies, thanks a lot !!!

Please sign in to comment.