Skip to content
This repository has been archived by the owner on Sep 24, 2023. It is now read-only.

Incompatible w/ Sentry post role refactor #3

Closed
ralish opened this issue Dec 14, 2015 · 1 comment · Fixed by #4
Closed

Incompatible w/ Sentry post role refactor #3

ralish opened this issue Dec 14, 2015 · 1 comment · Fixed by #4

Comments

@ralish
Copy link
Contributor

ralish commented Dec 14, 2015

A huge refactor of Sentry's handling of roles landed in early November: The Great Role Refactor. This breaks compatibility with Sentry releases after this commit, including the recently released 8.0.0-rc1.

Fortunately, the changes required are minimal. I forked the repository and made the minimal changes to get everything working with the refactor for our own Sentry server, but without preserving any backwards compatibility (I just needed everything working again quickly). You can view them here, but you may prefer something which preserves backwards compatibility with earlier Sentry releases?

Happy to open a pull request though if you're interested or want to build off of these changes.
@ralish ralish mentioned this issue Dec 14, 2015
Merged
@ralish
Copy link
Contributor Author

ralish commented Dec 14, 2015

Spent a little time cleaning up the changes to potentially be compatible with older Sentry versions (but see the PR notes!). Still, it's probably a decent start...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Role refactor fixes
1 participant
@ralish