-
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch routing from teams to product areas #126
Comments
@hubertdeng123 at getsentry/eng-pipes#436 (comment):
|
We could drop the table and recreate it in a migration. That wouldn't require ops intervention. It would mean starting from scratch with notifications, which probably isn't the end of the world since the change from teams to product owners is pretty significant already.
I think the answer is no. I think we should land getsentry/sentry#47156 asap, start manually moving tickets to the labels to get a sense for what might be missing, and then bring in the automation when we're ready. |
Deployed getsentry/eng-pipes#436! I set up a notification subscription in #feed-issues and tested routing with getsentry/sentry#48246. It failed. Gonna debug in getsentry/sentry#46554. Also a weird result🔒 from subscription:
|
Routing fixed in getsentry/eng-pipes#439. |
Chatted with @souredoutlook & Co. Not sure of the value here for old tickets. Need to think this out more. 🤔 |
I made https://github.com/getsentry/security-as-code/pull/71 to fill out remaining empty areas after talking with Zac. I need to get that merged and then set up notifications for all teams. I think I need to make one more PR to adjust membership for some perf-related areas. |
Notifications
|
Waiting for an automation to run to populate members of GitHub teams, then I can close this out. |
E.g. expecting two members here: https://github.com/orgs/getsentry/teams/product-owners-docs/members. |
Populated! |
😭 |
Move issues from "Team: Foo" to "Product Area: Foo"– puntingThe text was updated successfully, but these errors were encountered: