Consider merging

[ddavidebor]

Hello,
Please consider merging with frappe when you feel it's as stable as it should be!

David

[Alchez]

Hey @ddavidebor, thanks for reaching out!

Yes, we do plan to eventually contribute this integration to Frappe, but we're fairly shorthanded to take that up right now.

If you'd like to pick up the contribution for this, that would be okay too :)

Rohan

[ddavidebor]

What work is missing to be done?

David

[Alchez]

There isn't much functionality we will be adding anytime soon, so the things really to do here is to find an elegant way to integrate this as an alternative logger inside Frappe, along with proper documentation and tests.

[ddavidebor]

Perhaps the best way is not using an APP and instead of merging the changes with frappe itself. The dev team could hardcode their sentry info if they wanted, or anybody else could configure it through the system settings and/or environmental variables.

[Alchez]

Sure, I agree, but like I said, we don't have the bandwidth to take this up right now, and we wouldn't mind if someone from the community wanted to go ahead and merge it with Frappe.