Skip to content

Commit

Permalink
Merge pull request #292 from ameerul-deriv/FEQ-2610-no-result-shown-a…
Browse files Browse the repository at this point in the history
…fter-sell-order

Ameerul / FEQ-2610 Once the advertiser places a Sell order , and navigate to currency filter shows "No Result"
  • Loading branch information
ameerul-deriv authored Aug 22, 2024
2 parents 128e220 + 6f59ca0 commit 40582cf
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
1 change: 0 additions & 1 deletion src/hooks/api/settings/p2p-settings/useSettings.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,6 @@ const useSettings = () => {
return {
...rest,
data: p2pSettings,
setP2PSettings,
};
};

Expand Down
10 changes: 1 addition & 9 deletions src/routes/AppContent/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,7 @@ const AppContent = () => {

const [activeTab, setActiveTab] = useState(() => getActiveTab(location.pathname));
const [hasCreatedAdvertiser, setHasCreatedAdvertiser] = useState(false);
const {
isActive,
isLoading: isP2PSettingsLoading,
setP2PSettings,
subscribe: subscribeP2PSettings,
} = api.settings.useSettings();
const { isActive, isLoading: isP2PSettingsLoading, subscribe: subscribeP2PSettings } = api.settings.useSettings();
const {
error,
isActive: isSubscribed,
Expand All @@ -62,9 +57,6 @@ const AppContent = () => {
if (activeAccountData) {
subscribeP2PSettings({});
}
return () => {
setP2PSettings({});
};
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [activeAccountData]);

Expand Down

0 comments on commit 40582cf

Please sign in to comment.