Skip to content

Commit c2c67b9

Browse files
authored
FUI - Reports - Moving to native elements (#2721)
1 parent 62db7d9 commit c2c67b9

22 files changed

+153
-134
lines changed

src/apim.design.module.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ import { ProductSubscribeEditorModule as ProductSubscribeDesignModule } from "./
3131
import { ProductApisEditorModule as ProductApisDesignModule } from "./components/products/product-apis/ko/productApisEditor.module";
3232
import { ProductSubscriptionsEditorModule as ProductSubscriptionsDesignModule } from "./components/products/product-subscriptions/ko/productSubscriptionsEditor.module";
3333
import { App } from "./components/app/app";
34-
import { ReportsEditorModule as ReportsDesignModule } from "./components/reports/ko/reportsEditor.module";
34+
import { ReportsEditorModule as ReportsDesignModule } from "./components/reports/reports.design.module";
3535
import { ResetPasswordDesignModule } from "./components/users/reset-password/resetPassword.design.module";
3636
import { ConfirmPasswordDesignModule } from "./components/users/confirm-password/confirmPassword.design.module";
3737
import { HelpModule } from "./components/help";

src/apim.publish.module.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ import { IdentityService } from "./services/identityService";
2828
import { ResetPasswordPublishModule } from "./components/users/reset-password/resetPassword.publish.module";
2929
import { ConfirmPasswordPublishModule } from "./components/users/confirm-password/confirmPassword.publish.module";
3030
import { ChangePasswordPublishModule } from "./components/users/change-password/changePassword.publish.module";
31-
import { ReportsPublishModule } from "./components/reports/ko/reports.module";
31+
import { ReportsPublishModule } from "./components/reports/reports.publish.module";
3232
import { TenantService } from "./services/tenantService";
3333
import { ValidationSummaryPublishModule } from "./components/users/validation-summary/validationSummary.publish.module";
3434
import { BackendService } from "./services/backendService";

src/apim.runtime.module.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ import { ResetPasswordRuntimeModule } from "./components/users/reset-password/re
110110
import { ChangePasswordRuntimeModule } from "./components/users/change-password/changePassword.runtime.module";
111111
import { ConfirmPasswordRuntimeModule } from "./components/users/confirm-password/confirmPassword.runtime.module";
112112
import { SubscriptionsRuntimeModule } from "./components/users/subscriptions/subscriptions.runtime.module";
113-
import { ReportsRuntimeModule } from "./components/reports/ko/reports.runtime.module";
113+
import { ReportsRuntimeModule } from "./components/reports/reports.runtime.module";
114114
import { ValidationSummaryRuntimeModule } from "./components/users/validation-summary/validationSummary.runtime.module";
115115

116116
export class ApimRuntimeModule implements IInjectorModule {

src/components/react.components.d.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,5 +31,7 @@ declare namespace JSX {
3131
"fui-signin-runtime": any;
3232
"signin-runtime": any;
3333
"placeholder-content": any;
34+
"fui-reports-runtime": any;
35+
"reports-runtime": any;
3436
}
3537
}
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
import * as React from "react";
2+
import { LocalStyles } from "@paperbits/common/styles";
3+
4+
export class ReportsViewModel extends React.Component {
5+
public state: {
6+
isRedesignEnabled: boolean;
7+
styles: LocalStyles;
8+
};
9+
10+
constructor(props) {
11+
super(props);
12+
13+
this.state = {...props};
14+
}
15+
16+
public render(): JSX.Element {
17+
const data = JSON.stringify(this.state);
18+
19+
return this.state.isRedesignEnabled
20+
? <fui-reports-runtime props={data} ></fui-reports-runtime>
21+
: <reports-runtime params={data}></reports-runtime>;
22+
}
23+
}

src/components/reports/react/Charts/ApiCalls.tsx renamed to src/components/reports/react/runtime/Charts/ApiCalls.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import * as React from "react";
22
import { useCallback, useEffect, useState } from "react";
3-
import { Body1, Subtitle2Stronger } from "@fluentui/react-components";
43
import { IChartProps, LineChart } from "@fluentui/react-charting";
54
import { TReportsChartProps } from "../ReportsRuntime";
65

@@ -51,9 +50,9 @@ export const ApiCalls = ({ reportsByTime, timeRange, dateFormattingFunc }: TRepo
5150

5251
return (
5352
<div className={"report-chart-container"}>
54-
<Subtitle2Stronger block className={"report-chart-title"}>API Calls</Subtitle2Stronger>
53+
<h5 className={"report-chart-title"}>API Calls</h5>
5554
{reportsByTime.length === 0
56-
? <Body1 block>No data</Body1>
55+
? <div>No data</div>
5756
: <LineChart
5857
data={data}
5958
xAxisTitle="Time"

src/components/reports/react/Charts/ApiResponseTime.tsx renamed to src/components/reports/react/runtime/Charts/ApiResponseTime.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import * as React from "react";
22
import { useCallback, useEffect, useState } from "react";
3-
import { Body1, Subtitle2Stronger } from "@fluentui/react-components";
43
import { IChartProps, LineChart } from "@fluentui/react-charting";
5-
import { Utils } from "../../../../utils";
4+
import { Utils } from "../../../../../utils";
65
import { TReportsChartProps } from "../ReportsRuntime";
76

87
const apiResponseTimeLineChartData: IChartProps = {
@@ -47,9 +46,9 @@ export const ApiResponseTime = ({ reportsByTime, timeRange, dateFormattingFunc }
4746

4847
return (
4948
<div className={"report-chart-container"}>
50-
<Subtitle2Stronger block className={"report-chart-title"}>API Response Times</Subtitle2Stronger>
49+
<h5 className={"report-chart-title"}>API Response Times</h5>
5150
{reportsByTime.length === 0
52-
? <Body1 block>No data</Body1>
51+
? <div>No data</div>
5352
: <LineChart
5453
data={data}
5554
xAxisTitle="Time"

src/components/reports/react/Charts/DataTransfer.tsx renamed to src/components/reports/react/runtime/Charts/DataTransfer.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import * as React from "react";
22
import { useCallback, useEffect, useState } from "react";
3-
import { Body1, Subtitle2Stronger } from "@fluentui/react-components";
43
import { IChartProps, LineChart } from "@fluentui/react-charting";
5-
import { Utils } from "../../../../utils";
4+
import { Utils } from "../../../../../utils";
65
import { TReportsChartProps } from "../ReportsRuntime";
76

87
const dataTransferLineChartData: IChartProps = {
@@ -37,9 +36,9 @@ export const DataTransfer = ({ reportsByTime, timeRange, dateFormattingFunc }: T
3736

3837
return (
3938
<div className={"report-chart-container"}>
40-
<Subtitle2Stronger block className={"report-chart-title"}>Data Transfer</Subtitle2Stronger>
39+
<h5 className={"report-chart-title"}>Data Transfer</h5>
4140
{reportsByTime.length === 0
42-
? <Body1 block>No data</Body1>
41+
? <div>No data</div>
4342
: <LineChart
4443
data={data}
4544
xAxisTitle="Time"

src/components/reports/react/ReportsRuntime.tsx renamed to src/components/reports/react/runtime/ReportsRuntime.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@ import * as moment from "moment/moment";
44
import { Stack } from "@fluentui/react";
55
import { FluentProvider } from "@fluentui/react-components";
66
import { Resolve } from "@paperbits/react/decorators";
7-
import { AnalyticsService } from "../../../services/analyticsService";
8-
import { ReportRecordByTime } from "../../../contracts/reportRecordByTime";
9-
import * as Constants from "../../../constants";
7+
import { AnalyticsService } from "../../../../services/analyticsService";
8+
import { ReportRecordByTime } from "../../../../contracts/reportRecordByTime";
9+
import * as Constants from "../../../../constants";
1010
import {
1111
optionsReports,
1212
optionsReportsCharts,
File renamed without changes.

0 commit comments

Comments
 (0)