diff --git a/backend/code_review_backend/issues/api.py b/backend/code_review_backend/issues/api.py index 671f61e36..7d8de6824 100644 --- a/backend/code_review_backend/issues/api.py +++ b/backend/code_review_backend/issues/api.py @@ -213,7 +213,7 @@ def get_queryset(self): Issue.objects.filter(issue_links__diff=diff) .annotate(publishable=Q(issue_links__in_patch=True) & Q(level=LEVEL_ERROR)) .values( - "provider_id", + "id", "hash", "analyzer", "analyzer_check", diff --git a/backend/code_review_backend/issues/serializers.py b/backend/code_review_backend/issues/serializers.py index fd45005ff..d2d49fb45 100644 --- a/backend/code_review_backend/issues/serializers.py +++ b/backend/code_review_backend/issues/serializers.py @@ -191,6 +191,7 @@ class DiffFullSerializer(serializers.ModelSerializer): class Meta: model = Diff fields = ( + "id", "provider_id", "revision", "review_task_id", diff --git a/backend/code_review_backend/issues/tests/test_diff.py b/backend/code_review_backend/issues/tests/test_diff.py index 0779be28f..5c21be119 100644 --- a/backend/code_review_backend/issues/tests/test_diff.py +++ b/backend/code_review_backend/issues/tests/test_diff.py @@ -66,6 +66,7 @@ def test_list_diffs(self): "previous": None, "results": [ { + "id": 3, "provider_id": "PHID-DIFF-3", "revision": { "id": 1, @@ -96,6 +97,7 @@ def test_list_diffs(self): "created": self.now, }, { + "id": 2, "provider_id": "PHID-DIFF-2", "revision": { "id": 2, @@ -126,6 +128,7 @@ def test_list_diffs(self): "created": self.now, }, { + "id": 1, "provider_id": "PHID-DIFF-1", "revision": { "id": 1, diff --git a/frontend/src/Diffs.vue b/frontend/src/Diffs.vue index 5aa43c888..3c3e07816 100644 --- a/frontend/src/Diffs.vue +++ b/frontend/src/Diffs.vue @@ -156,7 +156,7 @@ export default { name: 'revision', params: { revisionId: diff.revision.id }, }" - >D{{ diff.revision.phabricator_id }}Revision {{ diff.revision.provider_id }} @ base: {{ diff.revision.base_repository | short_repo }} - head: {{ diff.revision.head_repository | short_repo }} @@ -217,10 +217,18 @@ export default {