openedx-unsupported / tutor-contrib-ralph

Tutor plugin for the Ralph LRS
Apache License 2.0
0 stars 1 forks source link

feat: add support for sentry #15

Closed Ian2012 closed 1 year ago

Ian2012 commented 1 year ago

Description

This PR adds support for sentry configuration variables

openedx-webhooks commented 1 year ago

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

Ian2012 commented 1 year ago

@jmaupetit what do you think?

Ian2012 commented 1 year ago

I wasn't sure if I could add those variables too, thanks I will address that

Ian2012 commented 1 year ago

Resolved

openedx-webhooks commented 1 year ago

@Ian2012 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

jmaupetit commented 1 year ago

Follow up to improve Sentry integration https://github.com/openfun/ralph/issues/322

bmtcril commented 1 year ago

Thanks @jmaupetit ! I'm watching that issue for future so we can upgrade when it lands.