Skip to content

Feature/mobile features#1049

Open
idormenco wants to merge 11 commits intomobilefrom
feature/mobile-features
Open

Feature/mobile features#1049
idormenco wants to merge 11 commits intomobilefrom
feature/mobile-features

Conversation

@idormenco
Copy link
Contributor

@idormenco idormenco commented Feb 2, 2026

No description provided.

@vercel
Copy link

vercel bot commented Feb 2, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
votemonitor Ready Ready Preview, Comment Feb 13, 2026 2:44pm

Request Review

@idormenco idormenco requested review from luciatugui and radulescuandrew and removed request for radulescuandrew February 3, 2026 07:51
@idormenco
Clean up commented code in index.tsx
e36c70a 
Removed commented-out code related to polling station information.
github-advanced-security[bot]
github-advanced-security bot found potential problems Feb 13, 2026
View reviewed changes
mobile/components/PollingStationGeneral.tsx
import SelectFormLanguageDialogContent from "./SelectFormLanguageDialogContent";
import { Typography } from "./Typography";
import { useState } from "react";
import { deleteSecureStoreItemAsync } from "../helpers/SecureStoreWrapper";

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused import deleteSecureStoreItemAsync.

Copilot Autofix

AI about 12 hours ago

In general, unused imports should be removed to keep the codebase clean and avoid confusion. They can also marginally improve bundle size and build performance.

For this specific case, the best fix is to delete the unused import of deleteSecureStoreItemAsync from ../helpers/SecureStoreWrapper in mobile/components/PollingStationGeneral.tsx. No other changes are required because there is no usage of that symbol anywhere in the component. This preserves all existing functionality while resolving the CodeQL warning.

Concretely:

  • In mobile/components/PollingStationGeneral.tsx, remove line 20 containing import { deleteSecureStoreItemAsync } from "../helpers/SecureStoreWrapper";.
  • No new methods, imports, or definitions are needed.
Suggested changeset 1
mobile/components/PollingStationGeneral.tsx

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/mobile/components/PollingStationGeneral.tsx b/mobile/components/PollingStationGeneral.tsx
--- a/mobile/components/PollingStationGeneral.tsx
+++ b/mobile/components/PollingStationGeneral.tsx
@@ -17,7 +17,6 @@
 import SelectFormLanguageDialogContent from "./SelectFormLanguageDialogContent";
 import { Typography } from "./Typography";
 import { useState } from "react";
-import { deleteSecureStoreItemAsync } from "../helpers/SecureStoreWrapper";
 
 interface PollingStationGeneralProps {
   psiData: PollingStationInformationAPIResponse | null | undefined;
EOF
@@ -17,7 +17,6 @@
import SelectFormLanguageDialogContent from "./SelectFormLanguageDialogContent";
import { Typography } from "./Typography";
import { useState } from "react";
import { deleteSecureStoreItemAsync } from "../helpers/SecureStoreWrapper";

interface PollingStationGeneralProps {
psiData: PollingStationInformationAPIResponse | null | undefined;
Copilot is powered by AI and may make mistakes. Always verify output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luciatugui luciatugui Awaiting requested review from luciatugui

@radulescuandrew radulescuandrew Awaiting requested review from radulescuandrew

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@idormenco