Browse Source

fix(dashboards): Add missing orderby values to table widgets (#64469)

These template table widgets were missing orderby values, so the order
was incorrect on the dashboard but correct in the widget builder.
Nar Saynorath 1 year ago
parent
commit
69336e3302

+ 3 - 3
static/app/views/dashboards/data.tsx

@@ -587,7 +587,7 @@ export const DASHBOARDS_TEMPLATES: DashboardTemplate[] = [
             aggregates: ['p75(measurements.lcp)'],
             columns: ['geo.country_code', 'geo.region'],
             conditions: 'has:geo.country_code',
-            orderby: '',
+            orderby: '-p75(measurements.lcp)',
           },
         ],
       },
@@ -1369,7 +1369,7 @@ export const DASHBOARDS_TEMPLATES: DashboardTemplate[] = [
             aggregates: ['p75(measurements.app_start_warm)'],
             columns: ['transaction'],
             conditions: 'has:measurements.app_start_warm',
-            orderby: '',
+            orderby: '-p75(measurements.app_start_warm)',
           },
         ],
       },
@@ -1393,7 +1393,7 @@ export const DASHBOARDS_TEMPLATES: DashboardTemplate[] = [
             aggregates: ['p75(measurements.app_start_cold)'],
             columns: ['transaction'],
             conditions: 'has:measurements.app_start_cold',
-            orderby: '',
+            orderby: '-p75(measurements.app_start_cold)',
           },
         ],
       },

+ 2 - 2
static/app/views/dashboards/widgetLibrary/data.tsx

@@ -87,7 +87,7 @@ export const getDefaultWidgets = () => {
           fields: ['session.status', 'sum(session)'],
           aggregates: ['sum(session)'],
           columns: ['session.status'],
-          orderby: '',
+          orderby: '-sum(session)',
         },
       ],
     },
@@ -105,7 +105,7 @@ export const getDefaultWidgets = () => {
           fields: ['geo.country_code', 'geo.region', 'p75(measurements.lcp)'],
           aggregates: ['p75(measurements.lcp)'],
           columns: ['geo.country_code', 'geo.region'],
-          orderby: '',
+          orderby: '-p75(measurements.lcp)',
         },
       ],
     },