Fixes missing actions on table, unused query parameter ?, and some aesthetics (opensearch-project#103)

dbbaughe · web-flow · commit 18c564c3a2a6 · 2020-07-31T14:55:54.000-07:00
diff --git a/index.js b/index.js
@@ -27,7 +27,7 @@ export default function (kibana) {
     name: "opendistro_index_management_kibana",
     uiExports: {
       app: {
-        title: "Index Management Kibana",
+        title: "Index Management",
         description: "Kibana plugin for Index Management",
         main: "plugins/opendistro_index_management_kibana/app",
         category: DEFAULT_APP_CATEGORIES.management,
diff --git a/public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx b/public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx
@@ -40,7 +40,7 @@ import _ from "lodash";
 import { ContentPanel, ContentPanelActions } from "../../../../components/ContentPanel";
 import ManagedIndexControls from "../../components/ManagedIndexControls";
 import ManagedIndexEmptyPrompt from "../../components/ManagedIndexEmptyPrompt";
-import { ACTIONS, DEFAULT_PAGE_SIZE_OPTIONS, DEFAULT_QUERY_PARAMS } from "../../utils/constants";
+import { DEFAULT_PAGE_SIZE_OPTIONS, DEFAULT_QUERY_PARAMS } from "../../utils/constants";
 import { BREADCRUMBS, DEFAULT_EMPTY_DATA, PLUGIN_NAME, ROUTES } from "../../../../utils/constants";
 import InfoModal from "../../components/InfoModal";
 import PolicyModal from "../../../../components/PolicyModal";
@@ -126,7 +126,9 @@ export default class ManagedIndices extends Component<ManagedIndicesProps, Manag
         truncateText: false,
         width: "150px",
         // @ts-ignore
-        render: (action: string) => ACTIONS[action] || DEFAULT_EMPTY_DATA,
+        render: (action: string) => (
+          <span style={{ textTransform: "capitalize" }}>{(action || DEFAULT_EMPTY_DATA).split("_").join(" ")}</span>
+        ),
       },
       {
         field: "managedIndexMetaData.info",
@@ -143,7 +145,7 @@ export default class ManagedIndices extends Component<ManagedIndicesProps, Manag
       },
       {
         field: "index", // we don't care about the field as we're using the whole item in render
-        name: "Status",
+        name: "Job Status",
         sortable: false,
         truncateText: false,
         width: "150px",
@@ -238,7 +240,7 @@ export default class ManagedIndices extends Component<ManagedIndicesProps, Manag
         if (failures) {
           toastNotifications.addDanger(
             `Failed to remove policy from ${failedIndices
-              .map(failedIndex => `[${failedIndex.indexName}, ${failedIndex.reason}]`)
+              .map((failedIndex) => `[${failedIndex.indexName}, ${failedIndex.reason}]`)
               .join(", ")}`
           );
         }
@@ -345,7 +347,7 @@ export default class ManagedIndices extends Component<ManagedIndicesProps, Manag
                 selectedItems.length === 1 ? `policy from ${selectedItems[0].index}` : `policies from ${selectedItems.length} indices`
               } permanently? This action cannot be undone.`,
               actionMessage: "Remove",
-              onAction: () => this.onClickRemovePolicy(selectedItems.map(item => item.index)),
+              onAction: () => this.onClickRemovePolicy(selectedItems.map((item) => item.index)),
             }),
         },
       },
diff --git a/public/pages/ManagedIndices/containers/ManagedIndices/__snapshots__/ManagedIndices.test.tsx.snap b/public/pages/ManagedIndices/containers/ManagedIndices/__snapshots__/ManagedIndices.test.tsx.snap
@@ -459,9 +459,9 @@ exports[`<ManagedIndices /> spec renders the component 1`] = `
                   >
                     <span
                       class="euiTableCellContent__text"
-                      title="Status"
+                      title="Job Status"
                     >
-                      Status
+                      Job Status
                     </span>
                   </div>
                 </th>
diff --git a/public/pages/ManagedIndices/models/interfaces.ts b/public/pages/ManagedIndices/models/interfaces.ts
@@ -23,15 +23,3 @@ export interface ManagedIndicesQueryParams {
   sortField: keyof ManagedIndexItem;
   sortDirection: Direction;
 }
-
-export interface Actions {
-  [action: string]: string;
-
-  rollover: string;
-  delete: string;
-  open: string;
-  close: string;
-  transition: string;
-  read_only: string;
-  read_write: string;
-}
diff --git a/public/pages/ManagedIndices/utils/constants.ts b/public/pages/ManagedIndices/utils/constants.ts
@@ -13,7 +13,7 @@
  * permissions and limitations under the License.
  */
 
-import { Actions, ManagedIndicesQueryParams } from "../models/interfaces";
+import { ManagedIndicesQueryParams } from "../models/interfaces";
 
 export const DEFAULT_PAGE_SIZE_OPTIONS = [5, 10, 20, 50];
 export const DEFAULT_QUERY_PARAMS: ManagedIndicesQueryParams = {
@@ -23,16 +23,3 @@ export const DEFAULT_QUERY_PARAMS: ManagedIndicesQueryParams = {
   sortField: "index",
   sortDirection: "desc",
 };
-
-export const ACTIONS: Actions = {
-  rollover: "Rollover",
-  delete: "Delete",
-  transition: "Transition",
-  open: "Open",
-  close: "Close",
-  read_only: "Read only",
-  read_write: "Read write",
-  replica_count: "Replica count",
-  notification: "Notification",
-  force_merge: "Force merge",
-};
diff --git a/public/services/IndexService.ts b/public/services/IndexService.ts
@@ -27,7 +27,8 @@ export default class IndexService {
   }
 
   getIndices = async (queryParamsString: string): Promise<ServerResponse<GetIndicesResponse>> => {
-    const url = `..${NODE_API._INDICES}?${queryParamsString}`;
+    let url = `..${NODE_API._INDICES}`;
+    if (queryParamsString) url += `?${queryParamsString}`;
     const response = (await this.httpClient.get(url)) as IHttpResponse<ServerResponse<GetIndicesResponse>>;
     return response.data;
   };
@@ -47,14 +48,12 @@ export default class IndexService {
   };
 
   searchPolicies = async (searchValue: string, source: boolean = false): Promise<ServerResponse<SearchResponse<any>>> => {
+    const str = searchValue.trim();
     const mustQuery = {
       query_string: {
         default_field: "policy.policy_id",
         default_operator: "AND",
-        query: `*${searchValue
-          .trim()
-          .split(" ")
-          .join("* *")}*`,
+        query: str ? `*${str.split(" ").join("* *")}*` : "*",
       },
     };
     const body = {
diff --git a/public/services/ManagedIndexService.ts b/public/services/ManagedIndexService.ts
@@ -36,9 +36,9 @@ export default class ManagedIndexService {
   };
 
   getManagedIndices = async (queryParamsString: string): Promise<ServerResponse<GetManagedIndicesResponse>> => {
-    const response = (await this.httpClient.get(`..${NODE_API.MANAGED_INDICES}?${queryParamsString}`)) as IHttpResponse<
-      ServerResponse<GetManagedIndicesResponse>
-    >;
+    let url = `..${NODE_API.MANAGED_INDICES}`;
+    if (queryParamsString) url += `?${queryParamsString}`;
+    const response = (await this.httpClient.get(url)) as IHttpResponse<ServerResponse<GetManagedIndicesResponse>>;
     return response.data;
   };
 
diff --git a/public/services/PolicyService.test.ts b/public/services/PolicyService.test.ts
@@ -45,7 +45,7 @@ describe("PolicyService spec", () => {
     await policyService.putPolicy(policy, policyId);
 
     expect(httpClientMock.put).toHaveBeenCalledTimes(1);
-    expect(httpClientMock.put).toHaveBeenCalledWith(`..${NODE_API.POLICIES}/${policyId}?`, policy);
+    expect(httpClientMock.put).toHaveBeenCalledWith(`..${NODE_API.POLICIES}/${policyId}`, policy);
   });
 
   it("calls delete policy nodejs route when calling deletePolicy", async () => {
diff --git a/public/services/PolicyService.ts b/public/services/PolicyService.ts
@@ -28,7 +28,8 @@ export default class PolicyService {
   }
 
   getPolicies = async (queryParamsString: string): Promise<ServerResponse<GetPoliciesResponse>> => {
-    const url = `..${NODE_API.POLICIES}?${queryParamsString}`;
+    let url = `..${NODE_API.POLICIES}`;
+    if (queryParamsString) url += `?${queryParamsString}`;
     const response = (await this.httpClient.get(url)) as IHttpResponse<ServerResponse<GetPoliciesResponse>>;
     return response.data;
   };
@@ -40,7 +41,8 @@ export default class PolicyService {
     primaryTerm?: number
   ): Promise<ServerResponse<PutPolicyResponse>> => {
     const queryParamsString = queryString.stringify({ seqNo, primaryTerm });
-    const url = `..${NODE_API.POLICIES}/${policyId}?${queryParamsString}`;
+    let url = `..${NODE_API.POLICIES}/${policyId}`;
+    if (queryParamsString) url += `?${queryParamsString}`;
     const response = (await this.httpClient.put(url, policy)) as IHttpResponse<ServerResponse<PutPolicyResponse>>;
     return response.data;
   };
diff --git a/server/services/IndexService.ts b/server/services/IndexService.ts
@@ -54,11 +54,9 @@ export default class IndexService {
         sortField: string;
         sortDirection: string;
       };
+      const str = search.trim();
       const params = {
-        index: `*${search
-          .trim()
-          .split(" ")
-          .join("* *")}*`,
+        index: str ? `*${str.split(" ").join("* *")}*` : "*",
         format: "json",
         s: `${sortField}:${sortDirection}`,
       };
@@ -128,7 +126,7 @@ export default class IndexService {
         response: {
           failures: addResponse.failures,
           updatedIndices: addResponse.updated_indices,
-          failedIndices: addResponse.failed_indices.map(failedIndex => ({
+          failedIndices: addResponse.failed_indices.map((failedIndex) => ({
             indexName: failedIndex.index_name,
             indexUuid: failedIndex.index_uuid,
             reason: failedIndex.reason,
diff --git a/server/utils/helpers.ts b/server/utils/helpers.ts
@@ -50,15 +50,13 @@ export function transformManagedIndexMetaData(metaData: ExplainAPIManagedIndexMe
 }
 
 export function getMustQuery<T extends string>(field: T, search: string): MatchAllQuery | QueryStringQuery<T> {
+  const str = search.trim();
   if (search.trim()) {
     return {
       query_string: {
         default_field: field,
         default_operator: "AND",
-        query: `*${search
-          .trim()
-          .split(" ")
-          .join("* *")}*`,
+        query: str ? `*${str.split(" ").join("* *")}*` : "*",
       },
     };
   }

Original file line number	Diff line number	Diff line change
@@ -459,9 +459,9 @@ exports[`<ManagedIndices /> spec renders the component 1`] = `
`459`	`459`	`>`
`460`	`460`	`<span`
`461`	`461`	`class="euiTableCellContent__text"`
`462`		`- title="Status"`
	`462`	`+ title="Job Status"`
`463`	`463`	`>`
`464`		`- Status`
	`464`	`+ Job Status`
`465`	`465`	`</span>`
`466`	`466`	`</div>`
`467`	`467`	`</th>`