Configure Max Size for Dashboards API & Minor UI Changes (opendistro-…

…for-elasticsearch#266)
zhongnansu · Dec 18, 2020 · e056f9f · e056f9f
1 parent e2788cc
commit e056f9f
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 26 deletions.
diff --git a/.../report_definitions/report_settings/__tests__/__snapshots__/report_settings.test.tsx.snap b/.../report_definitions/report_settings/__tests__/__snapshots__/report_settings.test.tsx.snap
@@ -1055,9 +1055,6 @@ exports[`<ReportSettings /> panel dashboard create from in-context 1`] = `
         </div>
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -2124,9 +2121,6 @@ exports[`<ReportSettings /> panel display errors on create 1`] = `
         </div>
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -2609,9 +2603,6 @@ exports[`<ReportSettings /> panel render component 1`] = `
         />
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -3112,9 +3103,6 @@ exports[`<ReportSettings /> panel render edit, dashboard source 1`] = `
         />
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -3615,9 +3603,6 @@ exports[`<ReportSettings /> panel render edit, saved search source 1`] = `
         />
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -4118,9 +4103,6 @@ exports[`<ReportSettings /> panel render edit, visualization source 1`] = `
         />
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -5180,9 +5162,6 @@ exports[`<ReportSettings /> panel saved search create from in-context 1`] = `
         </div>
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
@@ -6242,9 +6221,6 @@ exports[`<ReportSettings /> panel visualization create from in-context 1`] = `
         </div>
       </div>
     </div>
-    <div
-      class="euiSpacer euiSpacer--l"
-    />
   </div>
 </div>
 `;
diff --git a/kibana-reports/public/components/report_definitions/report_settings/report_settings.tsx b/kibana-reports/public/components/report_definitions/report_settings/report_settings.tsx
@@ -689,7 +689,6 @@ export function ReportSettings(props: ReportSettingProps) {
         />
         <EuiSpacer />
         {displayVisualReportsFormatAndMarkdown}
-        <EuiSpacer />
       </EuiPageContentBody>
     </EuiPageContent>
   );

diff --git a/kibana-reports/public/components/report_definitions/report_trigger/report_trigger.tsx b/kibana-reports/public/components/report_definitions/report_trigger/report_trigger.tsx
@@ -479,7 +479,7 @@ export function ReportTrigger(props: ReportTriggerProps) {
           error={'Invalid cron expression.'}
           labelAppend={
             <EuiText size="xs">
-              <EuiLink href="https://opendistro.github.io/for-elasticsearch-docs/docs/alerting/cron/" target="_blank">
+              <EuiLink href="https://opendistro.github.io/for-elasticsearch-docs/docs/alerting/cron/" target="_blank" external={true}>
                 Cron help
               </EuiLink>
             </EuiText>

diff --git a/kibana-reports/server/routes/reportSource.ts b/kibana-reports/server/routes/reportSource.ts
@@ -44,6 +44,7 @@ export default function (router: IRouter) {
         const params: RequestParams.Search = {
           index: '.kibana',
           q: 'type:dashboard',
+          size: DEFAULT_MAX_SIZE,
         };
         responseParams = params;
       } else if (request.params.reportSourceType === 'visualization') {
-Original file line number
+Diff line change
@@ Expand Up @@
             </div>
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             </div>
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             />
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             />
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             />
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             />
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             </div>
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;
@@ Expand Down Expand Up @@
             </div>
           </div>
         </div>
-        <div
-          class="euiSpacer euiSpacer--l"
-        />
       </div>
     </div>
     `;