Skip to content

Commit

Permalink
qa fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
daibhin committed Dec 19, 2024
1 parent 2916760 commit dceb226
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,14 +42,17 @@ export const errorTrackingDataNodeLogic = kea<errorTrackingDataNodeLogicType>([
...values.response,
results: results
// remove merged issues
.filter(({ id }) => !ids.includes(id))
.filter(({ id }) => !mergingIds.includes(id))
.map((issue) =>
// replace primary issue
mergedIssue.id === issue.id ? mergedIssue : issue
),
})
await api.errorTracking.mergeInto(primaryIssue.id, mergingIds)
actions.loadData(true)
try {
await api.errorTracking.mergeInto(primaryIssue.id, mergingIds)
} catch (e) {
actions.loadData(true)
}
}
},
assignIssue: async ({ id, assigneeId }) => {
Expand Down
12 changes: 6 additions & 6 deletions posthog/api/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -515,12 +515,12 @@ def register_grandfathered_environment_nested_viewset(
["team_id"],
)

# projects_router.register(
# r"error_tracking",
# error_tracking.ErrorTrackingGroupViewSet,
# "project_error_tracking",
# ["team_id"],
# )
projects_router.register(
r"error_tracking/issue",
error_tracking.ErrorTrackingIssueViewSet,
"project_error_tracking_issue",
["team_id"],
)

projects_router.register(
r"error_tracking/stack_frames",
Expand Down
2 changes: 1 addition & 1 deletion posthog/api/error_tracking.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class Meta:
fields = ["assignee", "status"]


class ErrorTrackingGroupViewSet(TeamAndOrgViewSetMixin, ForbidDestroyModel, viewsets.ModelViewSet):
class ErrorTrackingIssueViewSet(TeamAndOrgViewSetMixin, ForbidDestroyModel, viewsets.ModelViewSet):
scope_object = "INTERNAL"
queryset = ErrorTrackingIssue.objects.all()
serializer_class = ErrorTrackingIssueSerializer
Expand Down

0 comments on commit dceb226

Please sign in to comment.