Skip to content

Commit b0ce9d7

Browse files
committed
refactor: simplify isDev feature flag
1 parent 4796466 commit b0ce9d7

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

groups/script.js

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,7 @@ const QUERY_PARAM_KEY = {
2727
DEV_FEATURE_FLAG: 'dev',
2828
GROUP_SEARCH: 'name',
2929
};
30-
31-
const featureFlags = {
32-
isMultipleGroupSharingEnabled:
33-
getParamValueFromURL(QUERY_PARAM_KEY.DEV_FEATURE_FLAG) === 'true',
34-
};
30+
const isDev = getParamValueFromURL(QUERY_PARAM_KEY.DEV_FEATURE_FLAG) === 'true';
3531

3632
const handler = {
3733
set: (obj, prop, value) => {
@@ -66,15 +62,13 @@ const handler = {
6662
});
6763
break;
6864
case 'search':
69-
if (featureFlags.isMultipleGroupSharingEnabled) {
65+
if (isDev) {
7066
setParamValueInURL(QUERY_PARAM_KEY.GROUP_SEARCH, value);
7167
dataStore.filteredGroupsIds = getDiscordGroupIdsFromSearch(
7268
Object.values(dataStore.groups),
7369
value,
7470
);
75-
}
76-
// Start: Remove this code if removing Feature Flag for isMultipleGroupSharingEnabled
77-
else if (value === '') {
71+
} else if (value === '') {
7872
if (dataStore.groups == null) break;
7973
dataStore.filteredGroupsIds = Object.values(dataStore.groups).map(
8074
(group) => group.id,
@@ -85,7 +79,6 @@ const handler = {
8579
.filter((group) => group.title.toLowerCase().includes(search))
8680
.map((group) => group.id);
8781
}
88-
// End: Remove this code if removing Feature Flag for isMultipleGroupSharingEnabled
8982
obj[prop] = value;
9083
break;
9184
case 'isGroupCreationModalOpen':
@@ -129,9 +122,7 @@ const dataStore = new Proxy(
129122
userSelf: null,
130123
groups: null,
131124
filteredGroupsIds: null,
132-
search: featureFlags.isMultipleGroupSharingEnabled
133-
? getParamValueFromURL(QUERY_PARAM_KEY.GROUP_SEARCH)
134-
: '',
125+
search: isDev ? getParamValueFromURL(QUERY_PARAM_KEY.GROUP_SEARCH) : '',
135126
discordId: null,
136127
isCreateGroupModalOpen: false,
137128
},
@@ -195,7 +186,7 @@ const afterAuthentication = async () => {
195186
};
196187
return acc;
197188
}, {});
198-
if (featureFlags.isMultipleGroupSharingEnabled) {
189+
if (isDev) {
199190
dataStore.filteredGroupsIds = getDiscordGroupIdsFromSearch(
200191
Object.values(dataStore.groups),
201192
dataStore.search,
@@ -218,9 +209,7 @@ const bindGroupCreationButton = () => {
218209

219210
const bindSearchInput = () => {
220211
const searchInput = document.querySelector('.search__input');
221-
if (featureFlags.isMultipleGroupSharingEnabled) {
222-
searchInput.value = dataStore.search;
223-
}
212+
if (isDev) searchInput.value = dataStore.search;
224213
searchInput.addEventListener('input', (e) => {
225214
dataStore.search = e.target.value;
226215
});

0 commit comments

Comments
 (0)