Added Bugsnag Adaptor #20
Conversation
|
Thank you so much for the PR 🤩. We're adding the |
|
@stnguyen90 I would greatly appreciate if you can review this PR. Please let me know if additional changes are needed. |
|
@michizhou looks like there are merge conflicts. Would you please resolve them? |
|
@stnguyen90 I have resolved the merge conflicts. Please let me know if additional changes are needed. |
|
@michizhou thanks a lot for your contributions during Hacktoberfest 2022! The PR will be merged soon after some more reviews. Please reach out to me on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord. |
| $adapter = new Bugsnag($bugsnagKey ? $bugsnagKey : ''); | ||
| $logger = new Logger($adapter); | ||
| $response = $logger->addLog($log); | ||
| $this->assertEquals(200, $response); |
There was a problem hiding this comment.
I tried running this test and it passed, but I don't see any new errors in bugsnag. Were you able to see something in bugsnag?
There was a problem hiding this comment.
@stnguyen90 I didn't see anything in Bugsnag, as I don't recall sending any errors to Bugsnag. Do you know how we can send errors to Bugsnag to validate these changes? I'm willing to help in any way I can.
There was a problem hiding this comment.
@michizhou, you can sign up for a trial at BugSnag.
Closes appwrite/appwrite#4211