-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Add Get Suppliers dialog #10025
Add Get Suppliers dialog #10025
Conversation
}, | ||
callback: function(r) { | ||
console.log(r); | ||
if(r.message) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this code seems very similar to the earlier block. make it common!
dialog.fields_dict.search_type.$input.change( function() { | ||
var args = dialog.get_values(); | ||
if(!args) return; | ||
if (args.search_type === "Tag") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just use the depend_on
property in the field definition above!
Closing for now, pls reopen |
} | ||
} | ||
|
||
function LoadSuppliers(r) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
snake_case
:)
Adds a dialog box that gets called when using the new Get Suppliers button. Adds suppliers based on tag of supplier Type.
Needs frappe/frappe#3759 to work.