Kitti U. @ Ecosoft

Results 116 comments of Kitti U. @ Ecosoft

> I believe the predominant pattern for downstream extensions of upstream behaviour is to declare hooks. While I personally also prefer inheritance, in this case I agree that the overall...

A question though. When you say to create a new hook on upstream. In this case, it is on erpnext app not in the framework. Is it advisable to create...

In this case, so you think, the Bulk Transaction featurs/doctypes should be moving into Frappe instead of ERPNext. And so, ERPNext is probably first downstream app, and in its hook.py...

Okay I think I get your point. Will be back after I have cooked something.

@blaggacao thank you for your review. So I would call this a done. If there is anything else for this to get accepted?

@blaggacao here you go. Just provide the hooks point.

Thanks @blaggacao @ruthra-kumar

May be if you add a default outgoing Email Account and it should work. It works for me. ![Screenshot_20241021_092420_Gmail](https://github.com/user-attachments/assets/f2f21d6d-4838-441f-be45-e3099422ff02)

I thought for FW port it is correct like this, just pull the commit here. No?