fix: avoid dupe columns, clear group name from url on none (hypertrace#1269)

Author: aaron-steinfeld

projects/observability/src/pages/explorer/explorer-dashboard-builder.ts

@@ -522,9 +522,26 @@ export class ExplorerDashboardBuilder {
   protected getAttributesToExcludeFromUserDisplay(context: ExplorerGeneratedDashboardContext): Set<string> {
     switch (context) {
       case ObservabilityTraceType.Api:
-        return new Set(['protocol', 'apiName', 'statusCode', 'duration', 'startTime', 'calls']);
+        return new Set([
+          'protocol',
+          'serviceName',
+          'apiName',
+          'statusCode',
+          'apiTraceErrorSpanCount',
+          'duration',
+          'startTime',
+          'calls'
+        ]);
       case SPAN_SCOPE:
-        return new Set(['protocolName', 'displaySpanName', 'statusCode', 'duration', 'startTime']);
+        return new Set([
+          'protocolName',
+          'serviceName',
+          'displaySpanName',
+          'statusCode',
+          'errorCount',
+          'duration',
+          'startTime'
+        ]);
       default:
         return assertUnreachable(context);
     }

projects/observability/src/pages/explorer/explorer.component.ts

@@ -195,7 +195,12 @@ export class ExplorerComponent {
   private getGroupByQueryParams(groupBy?: GraphQlGroupBy): QueryParamObject {
     const key = groupBy?.keys[0];
     if (key === undefined) {
-      return {};
+      return {
+        // Clear existing selection
+        [ExplorerQueryParam.Group]: undefined,
+        [ExplorerQueryParam.OtherGroup]: undefined,
+        [ExplorerQueryParam.GroupLimit]: undefined
+      };
     }
 
     return {