Skip to content
This repository was archived by the owner on Apr 18, 2024. It is now read-only.

Commit 17404fa

Browse files
committed
refactor: OPTIC-429: Step 2 to avoid complexity in reading the PR
1 parent 0024ae8 commit 17404fa

File tree

10 files changed

+6
-12
lines changed

10 files changed

+6
-12
lines changed

src/components/DataManager/DataManager.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { Space } from "../Common/Space/Space";
88
import { Spinner } from "../Common/Spinner";
99
import { Tabs, TabsItem } from "../Common/Tabs/Tabs";
1010
import { FiltersSidebar } from "../Filters/FiltersSidebar/FilterSidebar";
11-
import { DataView } from "../MainView";
11+
import { DataView } from "../MainView/DataView/Table";
1212
import "./DataManager.styl";
1313
import { Toolbar } from "./Toolbar/Toolbar";
1414

src/components/Label/Label.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { FieldsButton } from "../Common/FieldsButton";
99
import { Icon } from "../Common/Icon/Icon";
1010
import { Resizer } from "../Common/Resizer/Resizer";
1111
import { Space } from "../Common/Space/Space";
12-
import { DataView } from "../MainView";
12+
import { DataView } from "../MainView/DataView/Table";
1313
import "./Label.styl";
1414

1515
const LabelingHeader = ({ SDK, onClick, isExplorerMode }) => {

src/components/MainView/index.js

-3
This file was deleted.

src/mixins/DataStore/DataStore.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { flow, getRoot, types } from "mobx-state-tree";
22
import { guidGenerator } from "../../utils/random";
33
import { isDefined } from "../../utils/utils";
44
import { DEFAULT_PAGE_SIZE, getStoredPageSize } from "../../components/Common/Pagination/Pagination";
5-
import { FF_DEV_1470, FF_LOPS_E_3, isFF } from "../../utils/feature-flags";
5+
import { FF_LOPS_E_3, isFF } from "../../utils/feature-flags";
66

77
const listIncludes = (list, id) => {
88
const index =
@@ -197,7 +197,7 @@ export const DataStore = (
197197

198198
self.loading = true;
199199

200-
if(interaction === "filter" || ((!isFF(FF_DEV_1470)) && interaction === "ordering") || ((!isFF(FF_DEV_1470)) && reload)) {
200+
if(interaction === "filter" || interaction === "ordering" || reload) {
201201
self.page = 1;
202202
} else if (reload || isDefined(pageNumber)) {
203203
if (self.page === 0)

src/stores/Tabs/tab.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import { TabFilter } from "./tab_filter";
1414
import { TabHiddenColumns } from "./tab_hidden_columns";
1515
import { TabSelectedItems } from "./tab_selected_items";
1616
import { History } from '../../utils/history';
17-
import { FF_DEV_1470, FF_LOPS_12, isFF } from "../../utils/feature-flags";
17+
import { FF_LOPS_12, isFF } from "../../utils/feature-flags";
1818
import { CustomJSON, StringOrNumberID, ThresholdType } from "../types";
1919
import { clamp } from "../../utils/helpers";
2020

@@ -253,7 +253,7 @@ export const Tab = types
253253
setType(type) {
254254
self.type = type;
255255
self.root.SDK.invoke("tabTypeChanged", { tab: self.id, type });
256-
self.save({ reload: isFF(FF_DEV_1470) });
256+
self.save({ reload: false });
257257
},
258258

259259
setTarget(target) {

src/utils/feature-flags.js

-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
// Outliner + Details
22
export const FF_DEV_1170 = "ff_front_1170_outliner_030222_short";
33

4-
// Switch to page navigation
5-
export const FF_DEV_1470 = "ff_front_dev_1470_dm_pagination_010422_short";
6-
74
/**
85
* Support for notification links in the Label Steam and the Review Stream.
96
* @link https://app.launchdarkly.com/default/branch/features/feat_front_dev_1752_notification_links_in_label_and_review_streams

0 commit comments

Comments
 (0)