Skip to content

✨ feat(aci): create issue alert migration helper for azure devops#83806

Merged
iamrajjoshi merged 1 commit into
raj/issue-alert-migration-helper-github-2from
raj/issue-alert-migration-helper-azure
Jan 28, 2025
Merged

✨ feat(aci): create issue alert migration helper for azure devops#83806
iamrajjoshi merged 1 commit into
raj/issue-alert-migration-helper-github-2from
raj/issue-alert-migration-helper-azure

Conversation

@iamrajjoshi

@iamrajjoshi iamrajjoshi commented Jan 22, 2025

Copy link
Copy Markdown
Collaborator

issue alert migration helper for azure devops, similar to github and github enterprise

branched off of #83793, will merge after this one is merged

closes https://getsentry.atlassian.net/browse/ACI-102

@iamrajjoshi iamrajjoshi self-assigned this Jan 22, 2025
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 22, 2025
@iamrajjoshi iamrajjoshi marked this pull request as ready for review January 22, 2025 15:57
@iamrajjoshi iamrajjoshi requested a review from a team as a code owner January 22, 2025 15:57
assert len(actions) == 0

def test_azure_devops_migration(self):
action_data = [

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#Todo for a future date -- it'd be a nice to have to have a self.create_notification_action or something along those lines to help create these

"""TicketDataBlob is a specific type that represents the data blob for a ticket creation action."""

# This is dynamic and can whatever customer config the customer setup on GitHub
dynamic_form_fields: list[dict] = field(default_factory=list)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: it might be nice to have stricter typing on list[dict] like what is the dict? TicketData?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will be part of a followup pr. i made the ticket here: https://getsentry.atlassian.net/browse/ACI-119

@iamrajjoshi iamrajjoshi merged commit c4c5dbf into raj/issue-alert-migration-helper-github-2 Jan 28, 2025
@iamrajjoshi iamrajjoshi deleted the raj/issue-alert-migration-helper-azure branch January 28, 2025 23:24
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants