Skip to content

feat: add button to support in modal blocked address #2884

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AGMASO wants to merge 3 commits into
base: main
Choose a base branch
from
+17 −0
Open

feat: add button to support in modal blocked address #2884

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e2ef31a
feat: add button to support in modal blocked address
AGMASO Feb 26, 2026
d442536
fix: adjustments css & text
AGMASO Feb 27, 2026
3a496b5
Update src/components/AddressBlockedModal.tsx
AGMASO Mar 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions src/components/AddressBlockedModal.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
import { ExclamationCircleIcon, LogoutIcon, RefreshIcon } from '@heroicons/react/outline';
import { Trans } from '@lingui/macro';
import { Box, Button, SvgIcon, Typography } from '@mui/material';
import { useRootStore } from 'src/store/root';
import { useShallow } from 'zustand/shallow';

import { BasicModal } from './primitives/BasicModal';

Expand All @@ -19,7 +21,17 @@ export const AddressBlockedModal = ({
}: AddressBlockedProps) => {
// eslint-disable-next-line @typescript-eslint/no-empty-function, @typescript-eslint/no-unused-vars
const setOpen = (_value: boolean) => {}; // ignore, we want the modal to not be dismissable
const [setFeedbackOpen, setSupportPrefillMessage] = useRootStore(
useShallow((state) => [state.setFeedbackOpen, state.setSupportPrefillMessage])
);
const handleGetSupport = () => {
const walletAddress = address ? address : 'Address not available';
const template = `Unable to Connect:\n\n"${walletAddress}"`;
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The support prefill template is hard-coded to Unable to Connect even when isError is true and the UI title shows Connection Error. Consider generating the template based on isError (and potentially including any available error context) so the message aligns with what the user is seeing.

Suggested change
const template = `Unable to Connect:\n\n"${walletAddress}"`;
const title = isError ? 'Connection Error' : 'Unable to Connect';
const template = `${title}:\n\n"${walletAddress}"`;

Copilot uses AI. Check for mistakes.

setSupportPrefillMessage(template);
setFeedbackOpen(true);
close();
Copy link

Copilot AI Feb 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The close() function is called but not defined. Unlike other modals in the codebase (such as Error.tsx), this component doesn't use the useModalContext hook to obtain the close function. Since AddressBlockedModal is designed to be non-dismissable (as indicated by the comment on line 23), calling close() here appears to be a copy-paste error from Error.tsx. This will cause a runtime error when the Get Support button is clicked. Consider removing the close() call, as the modal should remain open when the support feedback dialog opens.

Suggested change
close();

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

close() is called in handleGetSupport, but there is no close in scope (no prop and no useModalContext() hook), so this will fail to compile. Either import/use useModalContext (like TxErrorView does) and call its close, or remove the call if the blocked modal is intended to stay open while the Support modal opens.

Suggested change
close();

Copilot uses AI. Check for mistakes.
};
return (
<BasicModal open={true} withCloseButton={false} setOpen={setOpen}>
<Box
Expand Down Expand Up @@ -48,6 +60,10 @@ export const AddressBlockedModal = ({
<Trans>Refresh</Trans>
</Button>
)}

<Button variant="contained" onClick={handleGetSupport} size="small">
<Trans>Get support</Trans>
</Button>
<Button variant="contained" onClick={onDisconnectWallet}>
<SvgIcon fontSize="small" sx={{ mx: 1 }}>
<LogoutIcon />
Expand Down
1 change: 1 addition & 0 deletions src/locales/en/messages.po
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1065,6 +1065,7 @@ msgstr "Reactivate cooldown period to unstake {0} {stakedToken}"
msgid "Migrate to v3"
msgstr "Migrate to v3"

#: src/components/AddressBlockedModal.tsx
#: src/components/transactions/FlowCommons/Error.tsx
msgid "Get support"
msgstr "Get support"
Expand Down
Loading