Browse Source

fix(perf): Change sort on related issues (#67721)

Should be trends to better match new issues sort
Kev 11 months ago
parent
commit
b5c1af0d48

+ 1 - 1
static/app/views/performance/transactionSummary/transactionOverview/content.spec.tsx

@@ -89,7 +89,7 @@ describe('Transaction Summary Content', function () {
       body: [],
     });
     MockApiClient.addMockResponse({
-      url: '/organizations/org-slug/issues/?limit=5&query=is%3Aunresolved%20transaction%3Aexample-transaction&sort=new&statsPeriod=14d',
+      url: '/organizations/org-slug/issues/?limit=5&query=is%3Aunresolved%20transaction%3Aexample-transaction&sort=trends&statsPeriod=14d',
       body: [],
     });
     MockApiClient.addMockResponse({

+ 4 - 4
static/app/views/performance/transactionSummary/transactionOverview/index.spec.tsx

@@ -122,7 +122,7 @@ describe('Performance > TransactionSummary', function () {
       body: [],
     });
     MockApiClient.addMockResponse({
-      url: '/organizations/org-slug/issues/?limit=5&project=2&query=is%3Aunresolved%20transaction%3A%2Fperformance&sort=new&statsPeriod=14d',
+      url: '/organizations/org-slug/issues/?limit=5&project=2&query=is%3Aunresolved%20transaction%3A%2Fperformance&sort=trends&statsPeriod=14d',
       body: [],
     });
 
@@ -890,7 +890,7 @@ describe('Performance > TransactionSummary', function () {
 
     it('forwards conditions to related issues', async function () {
       const issueGet = MockApiClient.addMockResponse({
-        url: '/organizations/org-slug/issues/?limit=5&project=2&query=tag%3Avalue%20is%3Aunresolved%20transaction%3A%2Fperformance&sort=new&statsPeriod=14d',
+        url: '/organizations/org-slug/issues/?limit=5&project=2&query=tag%3Avalue%20is%3Aunresolved%20transaction%3A%2Fperformance&sort=trends&statsPeriod=14d',
         body: [],
       });
 
@@ -917,7 +917,7 @@ describe('Performance > TransactionSummary', function () {
 
     it('does not forward event type to related issues', async function () {
       const issueGet = MockApiClient.addMockResponse({
-        url: '/organizations/org-slug/issues/?limit=5&project=2&query=tag%3Avalue%20is%3Aunresolved%20transaction%3A%2Fperformance&sort=new&statsPeriod=14d',
+        url: '/organizations/org-slug/issues/?limit=5&project=2&query=tag%3Avalue%20is%3Aunresolved%20transaction%3A%2Fperformance&sort=trends&statsPeriod=14d',
         body: [],
         match: [
           (_, options) => {
@@ -1208,7 +1208,7 @@ describe('Performance > TransactionSummary', function () {
 
     it('uses MEP dataset for stats query and shows fallback warning', async function () {
       MockApiClient.addMockResponse({
-        url: '/organizations/org-slug/issues/?limit=5&project=2&query=has%3Anot-compatible%20is%3Aunresolved%20transaction%3A%2Fperformance&sort=new&statsPeriod=14d',
+        url: '/organizations/org-slug/issues/?limit=5&project=2&query=has%3Anot-compatible%20is%3Aunresolved%20transaction%3A%2Fperformance&sort=trends&statsPeriod=14d',
         body: [],
       });
       MockApiClient.addMockResponse({

+ 1 - 1
static/app/views/performance/transactionSummary/transactionOverview/relatedIssues.tsx

@@ -38,7 +38,7 @@ class RelatedIssues extends Component<Props> {
       end,
       statsPeriod,
       limit: 5,
-      sort: 'new',
+      sort: 'trends',
       ...pick(location.query, [...Object.values(URL_PARAM), 'cursor']),
     };
     const currentFilter = new MutableSearch(decodeScalar(location.query.query, ''));