diff --git a/.dockerignore b/.dockerignore index 26e4fd1..d340b4f 100644 --- a/.dockerignore +++ b/.dockerignore @@ -1,4 +1,3 @@ -data_sources/ .venv **/node_modules **/dist @@ -9,10 +8,3 @@ server-rs/target __pycache__ analyses/ *.log - -# Exclude data files except the ones we need -data/* -!data/wide.parquet -!data/filtered_uk_pois.parquet -!data/uk.pmtiles -!data/postcodes diff --git a/Dockerfile b/Dockerfile index dcaa83b..144b06c 100644 --- a/Dockerfile +++ b/Dockerfile @@ -20,12 +20,11 @@ WORKDIR /app COPY --from=server /app/server-rs/target/release/property-map-server ./ COPY --from=frontend /app/frontend/dist ./dist/ -# Copy data files into the image -COPY data/wide.parquet ./data/ -COPY data/filtered_uk_pois.parquet ./data/ -COPY data/uk.pmtiles ./data/ -COPY data/postcodes ./data/postcodes/ +# COPY property-data/wide.parquet ./data/ +# COPY property-data/filtered_uk_pois.parquet ./data/ +# COPY property-data/uk.pmtiles ./data/ +# COPY property-data/postcodes ./data/postcodes/ EXPOSE 8001 ENTRYPOINT ["./property-map-server"] -CMD ["--data", "/app/data/wide.parquet", "--pois", "/app/data/filtered_uk_pois.parquet", "--tiles", "/app/data/uk.pmtiles", "--postcodes", "/app/data/postcodes"] +CMD ["--data", "/app/data/wide.parquet", "--pois", "/app/data/filtered_uk_pois.parquet", "--tiles", "/app/data/uk.pmtiles", "--postcodes", "/app/data/new_postcode_boundaries"] diff --git a/README.md b/README.md index 30c38f8..a0397e9 100644 --- a/README.md +++ b/README.md @@ -60,17 +60,13 @@ We give you all the data and tools to become an Well-informed Buyer through the - scraping - fix frontend - - price history - map hexagons - dragging - account management - stripe -- saved views - update texts -- friendlier filtering - fix plausible - move data to raid -- loading animation on right pane - extract all user-facing texts into a yaml file for easy editing - register domain @@ -90,52 +86,33 @@ FAQ: ## outstadning prompts -Ensure the website is dynamically scalable and looks great on mobiles. The home/data sources/faq pages should be easy, just ensure responsive scaling. For the dashboard, we need a different layout. Split the screen vertically into 60:40. The top half is the map while the bottom is the filter section. Add a menu drawer on the right hand side where the area/properties/poi tabs will live. When the user clicks on a hexagon, the drawer appears covering 90% of the screen. Theres a large X button to dismiss it but clicking outside works too. +Add licensing to the app. By default, anonymous users can use the map but only in central london. if they try zooming out, the server refuses to provide data and the users will be prompted to buy a lifetime license to continue (or zoom back in). Just before buying a license, they have to register by providing their email address and password, then they need to complete the stripe check out workflow. Implement the full pocketbase/server/frontend integration. For admins, give an option to generate an invite link, opening which prompts you to register and gives you a free license forever. Have a cool animation with party poppers on the successful acquiring of a license. For non-admin users, allow inviting friends for 30% off the price. Also add a support page that shows my email address, and add a FAQ on the same page too. While doing this, protect the server against DOS-ing. + + +- +- the area stastics are missing for postcodes, they only work for hexagons +- in the mobile view, move the property density and previewing colour spectrum to the bottom half of the screen. +- make the no active filters have less padding on phone +- add blue/green rollout +- rename OgImageQuery to ScreenshotQuery +- make the eye and plus icons and their touch targets twice the size -Look at the frontend. Once you're logged in, you can save your searches and then look at your saved searches. Add a save search button for logged in users in the header. Also add a saved searches page which shows you the time of saving, the filters selected and a screenshot of the map. Use a similar approach we use to generate og images. -Add licensing to the app. By default, anonymous users can use the map but only in central london. if they try zooming out, the server refuses to provide data and the users will be prompted to buy a lifetime license to continue (or zoom back in). Just before buying a license, they have to register by providing their email address and password, then they need to complete the stripe check out workflow. Implement the full pocketbase/server/frontend integration. For admins, give an option to generate an invite link, opening which prompts you to register and gives you a free license forever. Have a cool animation with party poppers on the successful acquiring of a license. For non-admin users, allow inviting friends for 30% off the price. Also add a support page that shows my email address, and add a FAQ on the same page too. While doing this, protect the server against DOS-ing. - -- put price history under property and on the top of the tab add a description for the histograms - -- Can we embedd google street view in an iframe? If so, add it underneath the area page - -- You can reach out to ollama at http://ollama:11434/v1. Add a server endpoint that takes the selected filters and results of them shown on the frontend and generates a brief description of the area using qwen3:14b. Make sure that loading the llm summary doesn't block showing anything on the screen and instead add a placeholder animation in the Area tab where this functionality belongs. - -- Ensure that the Area & Properties tabs have a loading animation when waiting for data to be fetched - -- Add a forgot password workflow for the login page relying on pocketbase - -- Make sure the frontend asks for 25% more area than what the user sees on their viewport when fetching hexagons and postcodes to ensure a smoother panning experience. - -- The area tab is suppoed to show links to zoopla, rightmove, etc. but it doesn't. Investigate the issue - -- The colours of the log in popup are not consistent with the rest of the website, fix it. - -- Show a register button in the header too and make it green like the other buttons - -- Don't support profile pictures or full names to avoid GDPR requirements. - -- the min: number; max: number; p1: number; p99: number; counts: number[] histogram schema doesn't make sense. Have on the min max, width and counts in the schema but ensure min corresponds with the center of the bin and max with the center of the last bin - -- move the FEATURE_FORMATS definition to the backend's features.rs -wide = wide.with_columns( - ( - pl.col("date_of_transfer").dt.year() - + (pl.col("date_of_transfer").dt.month() - 1) / 12 - ) - .cast(pl.Float32) - .alias("transaction_year"), - ) - -is ugly, just put the actual datetime inside the parquet and call the column date of last transaction - -- make the text on the map white when in dark mode - -- move download_map_assets.py into the download module where all the other downloads live. Also, try to find a different way of downloading all icons without hardcoding them +## name ideas + + +perfect postcodes + +golden postcodes + +calculated move + +the spec + +geologic diff --git a/Taskfile.data.yml b/Taskfile.data.yml index 5118d7f..9e4958d 100644 --- a/Taskfile.data.yml +++ b/Taskfile.data.yml @@ -28,7 +28,7 @@ vars: INSPIRE_OUTPUT: "{{.DATA_DIR}}/inspire" OA_BOUNDARIES_OUTPUT: "{{.DATA_DIR}}/oa_boundaries.gpkg" UPRN_LOOKUP_OUTPUT: "{{.DATA_DIR}}/uprn_lookup.parquet" - POSTCODE_BOUNDARIES_OUTPUT: "{{.DATA_DIR}}/postcode_boundaries" + POSTCODE_BOUNDARIES_OUTPUT: "{{.DATA_DIR}}/new_postcode_boundaries" tasks: download:tiles: @@ -247,8 +247,6 @@ tasks: - download:oa-boundaries - download:inspire - download:uprn-lookup - status: - - test -d {{.POSTCODE_BOUNDARIES_OUTPUT}}/units cmds: - >- uv run python -m pipeline.transform.postcode_boundaries diff --git a/frontend/src/App.tsx b/frontend/src/App.tsx index 3534056..86e9a6d 100644 --- a/frontend/src/App.tsx +++ b/frontend/src/App.tsx @@ -13,6 +13,7 @@ import { fetchWithRetry, apiUrl } from './lib/api'; import { parseUrlState } from './lib/url-state'; import { INITIAL_VIEW_STATE } from './lib/consts'; import { useTheme } from './hooks/useTheme'; +import { useIsMobile } from './hooks/useIsMobile'; import { useAuth } from './hooks/useAuth'; import { useSavedSearches } from './hooks/useSavedSearches'; @@ -87,6 +88,7 @@ export default function App() { }); const { theme, toggleTheme } = useTheme(); + const isMobile = useIsMobile(); const { user, loading: authLoading, @@ -207,7 +209,7 @@ export default function App() { } return ( -
+
{activePage === 'home' ? ( navigateTo('dashboard')} theme={theme} /> @@ -257,6 +260,7 @@ export default function App() { onClearPendingInfoFeature={() => setPendingInfoFeature(null)} onNavigateTo={navigateTo} onExportStateChange={setExportState} + isMobile={isMobile} /> )} {showAuthModal && ( diff --git a/frontend/src/components/faq/FAQPage.tsx b/frontend/src/components/faq/FAQPage.tsx index 4b21dce..5b57fcf 100644 --- a/frontend/src/components/faq/FAQPage.tsx +++ b/frontend/src/components/faq/FAQPage.tsx @@ -65,7 +65,7 @@ const FAQ_ITEMS: FAQItem[] = [ { question: 'Does this work on mobile?', answer: - 'The app is designed for desktop browsers where you have enough screen space for the map, filter panel, and POI/properties panel side by side. It will load on mobile but the experience is best on a larger screen.', + 'Yes. On mobile, the dashboard uses a vertical split layout with the map on top and a tabbed panel below for filters, area stats, properties, and POIs. Tapping a hexagon opens a full-screen drawer with the details. The full desktop experience with side-by-side panels is available on screens 768px and wider.', }, ]; diff --git a/frontend/src/components/home/HomePage.tsx b/frontend/src/components/home/HomePage.tsx index 32101bb..219c26c 100644 --- a/frontend/src/components/home/HomePage.tsx +++ b/frontend/src/components/home/HomePage.tsx @@ -40,7 +40,7 @@ export default function HomePage({
{/* Hero */} -
+
Find where to live, not just what's for sale

-

+

Every neighbourhood
in England & Wales. @@ -158,7 +158,7 @@ export default function HomePage({
{STATS.map((s) => (
-
{s.value}
+
{s.value}
{s.label}
))} diff --git a/frontend/src/components/map/AreaPane.tsx b/frontend/src/components/map/AreaPane.tsx index a7855bb..49aa1db 100644 --- a/frontend/src/components/map/AreaPane.tsx +++ b/frontend/src/components/map/AreaPane.tsx @@ -10,7 +10,7 @@ import StackedBarChart from './StackedBarChart'; import StackedEnumChart from './StackedEnumChart'; import PriceHistoryChart from './PriceHistoryChart'; import ExternalSearchLinks from './ExternalSearchLinks'; -import { InfoIcon, CloseIcon } from '../ui/icons'; +import { InfoIcon, CloseIcon, ChevronIcon } from '../ui/icons'; import { CollapsibleGroupHeader } from '../ui/CollapsibleGroupHeader'; import { LightbulbIcon } from '../ui/icons/LightbulbIcon'; import { IconButton } from '../ui/IconButton'; @@ -58,6 +58,7 @@ export default function AreaPane({ const featureGroups = useMemo(() => groupFeaturesByCategory(globalFeatures), [globalFeatures]); const [infoFeature, setInfoFeature] = useState(null); const [collapsedGroups, setCollapsedGroups] = useState>(new Set()); + const [aiSummaryExpanded, setAiSummaryExpanded] = useState(true); const toggleGroup = (name: string) => setCollapsedGroups((prev) => { @@ -133,41 +134,53 @@ export default function AreaPane({ )} - {/* AI Summary Card */} - {(aiSummary || aiSummaryLoading || aiSummaryError) && ( -
-
-
- - AI Summary -
- {aiSummaryError ? ( -
- Failed to generate summary. - {onRetryAiSummary && ( - - )} -
- ) : aiSummaryLoading ? ( -
-
-
-
- ) : ( -

- {aiSummary} -

- )} -
-
- )} -
+ {/* AI Summary Card */} + {(aiSummary || aiSummaryLoading || aiSummaryError) && ( +
+
+ + {aiSummaryExpanded && ( + <> + {aiSummaryError ? ( +
+ Failed to generate summary. + {onRetryAiSummary && ( + + )} +
+ ) : aiSummaryLoading ? ( +
+
+
+
+ ) : ( +

+ {aiSummary} +

+ )} + + )} +
+
+ )} {loading && !stats ? ( ) : stats ? ( diff --git a/frontend/src/components/map/Filters.tsx b/frontend/src/components/map/Filters.tsx index 9dd777b..c4768ec 100644 --- a/frontend/src/components/map/Filters.tsx +++ b/frontend/src/components/map/Filters.tsx @@ -88,10 +88,10 @@ function FeatureBrowser({ return ( <> -
+
-
+
{grouped.map((group) => { const isExpanded = isSearching || expandedGroups.has(group.name); return ( @@ -187,7 +187,7 @@ export default memo(function Filters({ const [activeInfoFeature, setActiveInfoFeature] = useState(null); return ( -
+
-
+
@@ -215,7 +215,7 @@ export default memo(function Filters({
-
+
{enabledFeatureList.length === 0 && ( } @@ -308,11 +308,11 @@ export default memo(function Filters({
-
+
Add Filter
-
+
void; + bounds?: Bounds | null; } @@ -114,6 +117,7 @@ export default memo(function Map({ filters = {}, searchedPostcode, onPostcodeSearched, + bounds: viewportBounds, }: MapProps) { const containerRef = useRef(null); const [viewState, setViewState] = useState(initialViewState || INITIAL_VIEW_STATE); @@ -199,13 +203,18 @@ export default memo(function Map({ let min = Infinity; let max = -Infinity; for (const d of data) { + if (viewportBounds) { + const [lat, lng] = cellToLatLng(d.h3); + if (lat < viewportBounds.south || lat > viewportBounds.north || lng < viewportBounds.west || lng > viewportBounds.east) continue; + } const c = d.count as number; if (c < min) min = c; if (c > max) max = c; } + if (min === Infinity) return { min: 0, max: 1 }; if (min === max) return { min, max: min + 1 }; return { min, max }; - }, [data]); + }, [data, viewportBounds]); const colorFeatureMeta = useMemo( () => (viewFeature ? features.find((f) => f.name === viewFeature) || null : null), @@ -259,13 +268,18 @@ export default memo(function Map({ let min = Infinity; let max = -Infinity; for (const d of postcodeData) { + if (viewportBounds) { + const [lng, lat] = d.properties.centroid as [number, number]; + if (lat < viewportBounds.south || lat > viewportBounds.north || lng < viewportBounds.west || lng > viewportBounds.east) continue; + } const c = d.properties.count; if (c < min) min = c; if (c > max) max = c; } + if (min === Infinity) return { min: 0, max: 1 }; if (min === max) return { min, max: min + 1 }; return { min, max }; - }, [postcodeData]); + }, [postcodeData, viewportBounds]); const postcodeCountRangeRef = useRef(postcodeCountRange); postcodeCountRangeRef.current = postcodeCountRange; @@ -324,7 +338,7 @@ export default memo(function Map({ const cfm = colorFeatureMetaRef.current; const dark = isDarkRef.current; if (vf && clr && cfm) { - const val = d[`min_${vf}`]; + const val = d[`avg_${vf}`] ?? d[`min_${vf}`]; if (val == null) return (dark ? [80, 70, 65, 80] : [128, 128, 128, 80]) as [number, number, number, number]; if (fr) { const minVal = d[`min_${vf}`] as number; @@ -392,7 +406,7 @@ export default memo(function Map({ const cfm = colorFeatureMetaRef.current; const dark = isDarkRef.current; if (vf && clr && cfm) { - const val = d[`min_${vf}`]; + const val = d[`avg_${vf}`] ?? d[`min_${vf}`]; if (val == null) return (dark ? [80, 70, 65, 80] : [128, 128, 128, 80]) as [number, number, number, number]; if (fr) { const minVal = d[`min_${vf}`] as number; @@ -402,15 +416,15 @@ export default memo(function Map({ } } const range = clr[1] - clr[0]; - if (range === 0) return [...GRADIENT[0].color, 255] as [number, number, number, number]; + if (range === 0) return [...GRADIENT[0].color, 180] as [number, number, number, number]; const t = ((val as number) - clr[0]) / range; const rgb = normalizedToColor(Math.max(0, Math.min(1, t))); - return [...rgb, 255] as [number, number, number, number]; + return [...rgb, 180] as [number, number, number, number]; } const cr = postcodeCountRangeRef.current; const c = d.count; const t = (c - cr.min) / (cr.max - cr.min); - return [...countToColor(Math.max(0, Math.min(1, t)), densityGradientRef.current), 255] as [ + return [...countToColor(Math.max(0, Math.min(1, t)), densityGradientRef.current), 180] as [ number, number, number, @@ -442,6 +456,8 @@ export default memo(function Map({ pickable: true, onClick: handlePostcodeClick, onHover: handlePostcodeHoverCallback, + // @ts-expect-error beforeId is a MapboxOverlay interleave prop, not typed in LayerProps + beforeId: 'landuse_park', }), [postcodeData, postcodeColorTrigger, handlePostcodeClick, handlePostcodeHoverCallback] ); @@ -572,22 +588,9 @@ export default memo(function Map({ ) : ( <> - {viewSource === 'eye' && viewFeature && ( -
- - Previewing “{viewFeature}” - - -
- )} {viewFeature && colorRange && colorFeatureMeta ? ( {mode === 'density' ? ( <> - Few - Many + {formatValue(range[0])} + {formatValue(range[1])} ) : enumValues && enumValues.length > 0 ? ( <> diff --git a/frontend/src/components/map/MapPage.tsx b/frontend/src/components/map/MapPage.tsx index bd2698d..59e8193 100644 --- a/frontend/src/components/map/MapPage.tsx +++ b/frontend/src/components/map/MapPage.tsx @@ -7,6 +7,7 @@ import Filters from './Filters'; import POIPane from './POIPane'; import { PropertiesPane } from './PropertiesPane'; import AreaPane from './AreaPane'; +import MobileDrawer from './MobileDrawer'; import DataSources from '../data-sources/DataSources'; import { TabButton } from '../ui/TabButton'; import { useMapData } from '../../hooks/useMapData'; @@ -24,6 +25,8 @@ export interface ExportState { exporting: boolean; } +type MobileBottomTab = 'filters' | 'pois'; + interface MapPageProps { features: FeatureMeta[]; poiCategoryGroups: POICategoryGroup[]; @@ -39,6 +42,7 @@ interface MapPageProps { onExportStateChange?: (state: ExportState) => void; screenshotMode?: boolean; ogMode?: boolean; + isMobile?: boolean; } export default function MapPage({ @@ -56,6 +60,7 @@ export default function MapPage({ onExportStateChange, screenshotMode, ogMode, + isMobile = false, }: MapPageProps) { const [searchedPostcode, setSearchedPostcode] = useState(null); const [selectedPOICategories, setSelectedPOICategories] = useState>(initialPOICategories); @@ -63,6 +68,10 @@ export default function MapPage({ const [leftPaneWidth, leftPaneHandlers] = usePaneResize(384, 200, 600, 'left'); const [rightPaneWidth, rightPaneHandlers] = usePaneResize(288, 200, 500, 'right'); + // Mobile state + const [mobileBottomTab, setMobileBottomTab] = useState('filters'); + const [mobileDrawerOpen, setMobileDrawerOpen] = useState(false); + // Initialize filters first const { filters, @@ -123,6 +132,15 @@ export default function MapPage({ selection.setRightPaneTab(initialTab); }, []); // eslint-disable-line react-hooks/exhaustive-deps + // On mobile, open drawer and switch tab when hexagon is clicked + const handleMobileHexagonClick = useCallback((id: string, isPostcode?: boolean) => { + selection.handleHexagonClick(id, isPostcode); + if (id) { + setMobileDrawerOpen(true); + setMobileBottomTab('area'); + } + }, [selection.handleHexagonClick]); // eslint-disable-line react-hooks/exhaustive-deps + // Compute hexagon location for external links const hexagonLocation = useMemo(() => { const hexId = selection.selectedHexagon?.id; @@ -194,7 +212,7 @@ export default function MapPage({ if (screenshotMode) { return ( -
+
); } + // Shared pane content renderers + const renderAreaPane = () => ( + f.properties.postcode === selection.selectedHexagon?.id) || null + : null + } + onViewProperties={selection.handleViewPropertiesFromArea} + onClose={selection.handleCloseSelection} + hexagonLocation={hexagonLocation} + filters={filters} + onNavigateToSource={(slug, featureName) => onNavigateTo('data-sources', slug, featureName)} + aiSummary={aiSummary.summary} + aiSummaryLoading={aiSummary.loading} + aiSummaryError={aiSummary.error} + onRetryAiSummary={aiSummary.retry} + /> + ); + + const renderPropertiesPane = () => ( + onNavigateTo('data-sources', slug)} + /> + ); + + const renderPOIPane = () => ( + onNavigateTo('data-sources', slug)} + /> + ); + + const renderFilters = () => ( + onNavigateTo('data-sources', slug, featureName)} + openInfoFeature={pendingInfoFeature} + onClearOpenInfoFeature={onClearPendingInfoFeature} + /> + ); + + // Mobile layout + if (isMobile) { + return ( +
+ {initialLoading && ( +
+
+ +

Connecting to server...

+
+
+ )} + + {/* Map — 45% */} +
+ + {mapData.loading && ( +
+ Loading... +
+ )} + onNavigateTo('data-sources')} /> +
+ + {/* Bottom panel — 55% */} +
+ {/* Tab bar */} +
+ setMobileBottomTab('filters')} /> + setMobileBottomTab('pois')} /> +
+ + {/* Tab content */} +
+ {mobileBottomTab === 'pois' ? ( +
+ {renderPOIPane()} +
+ ) : ( + renderFilters() + )} +
+
+ + {/* Mobile drawer for full-screen hexagon details */} + {mobileDrawerOpen && selection.selectedHexagon && ( + setMobileDrawerOpen(false)} + renderArea={renderAreaPane} + renderProperties={renderPropertiesPane} + renderPOIs={renderPOIPane} + /> + )} +
+ ); + } + + // Desktop layout (unchanged) return (
{initialLoading && ( @@ -234,28 +405,7 @@ export default function MapPage({ {/* Left Pane */}
- onNavigateTo('data-sources', slug, featureName)} - openInfoFeature={pendingInfoFeature} - onClearOpenInfoFeature={onClearPendingInfoFeature} - /> + {renderFilters()}
{mapData.loading && (
@@ -314,45 +465,11 @@ export default function MapPage({
{selection.rightPaneTab === 'area' ? ( - f.properties.postcode === selection.selectedHexagon?.id) || null - : null - } - onViewProperties={selection.handleViewPropertiesFromArea} - onClose={selection.handleCloseSelection} - hexagonLocation={hexagonLocation} - filters={filters} - onNavigateToSource={(slug, featureName) => onNavigateTo('data-sources', slug, featureName)} - aiSummary={aiSummary.summary} - aiSummaryLoading={aiSummary.loading} - aiSummaryError={aiSummary.error} - onRetryAiSummary={aiSummary.retry} - /> + renderAreaPane() ) : selection.rightPaneTab === 'properties' ? ( - onNavigateTo('data-sources', slug)} - /> + renderPropertiesPane() ) : ( - onNavigateTo('data-sources', slug)} - /> + renderPOIPane() )}
diff --git a/frontend/src/components/map/MobileDrawer.tsx b/frontend/src/components/map/MobileDrawer.tsx new file mode 100644 index 0000000..5e5f5cc --- /dev/null +++ b/frontend/src/components/map/MobileDrawer.tsx @@ -0,0 +1,59 @@ +import { useState, useEffect } from 'react'; +import { CloseIcon } from '../ui/icons/CloseIcon'; +import { TabButton } from '../ui/TabButton'; + +type DrawerTab = 'area' | 'properties' | 'pois'; + +interface MobileDrawerProps { + onClose: () => void; + renderArea: () => React.ReactNode; + renderProperties: () => React.ReactNode; + renderPOIs: () => React.ReactNode; +} + +export default function MobileDrawer({ + onClose, + renderArea, + renderProperties, + renderPOIs, +}: MobileDrawerProps) { + const [tab, setTab] = useState('area'); + + // Close on Escape + useEffect(() => { + const handler = (e: KeyboardEvent) => { + if (e.key === 'Escape') onClose(); + }; + window.addEventListener('keydown', handler); + return () => window.removeEventListener('keydown', handler); + }, [onClose]); + + return ( +
+ {/* Backdrop — top 10% */} +
+ + {/* Panel — bottom 90% */} +
+ {/* Tab bar + close */} +
+ setTab('area')} /> + setTab('properties')} /> + setTab('pois')} /> + +
+ + {/* Content */} +
+ {tab === 'area' ? renderArea() : tab === 'properties' ? renderProperties() : renderPOIs()} +
+
+
+ ); +} diff --git a/frontend/src/components/ui/AuthModal.tsx b/frontend/src/components/ui/AuthModal.tsx index 3dd1548..47d786a 100644 --- a/frontend/src/components/ui/AuthModal.tsx +++ b/frontend/src/components/ui/AuthModal.tsx @@ -15,7 +15,7 @@ export default function AuthModal({ }: { onClose: () => void; onLogin: (email: string, password: string) => Promise; - onRegister: (email: string, password: string, name?: string) => Promise; + onRegister: (email: string, password: string) => Promise; onForgotPassword: (email: string) => Promise; loading: boolean; error: string | null; @@ -25,7 +25,6 @@ export default function AuthModal({ const [view, setView] = useState(initialTab); const [email, setEmail] = useState(''); const [password, setPassword] = useState(''); - const [name, setName] = useState(''); const [resetSent, setResetSent] = useState(false); const switchView = useCallback( @@ -45,7 +44,7 @@ export default function AuthModal({ await onLogin(email, password); onClose(); } else if (view === 'register') { - await onRegister(email, password, name || undefined); + await onRegister(email, password); onClose(); } else { await onForgotPassword(email); @@ -55,7 +54,7 @@ export default function AuthModal({ // Error is handled by the hook } }, - [view, email, password, name, onLogin, onRegister, onForgotPassword, onClose] + [view, email, password, onLogin, onRegister, onForgotPassword, onClose] ); const title = @@ -107,21 +106,6 @@ export default function AuthModal({ {/* Form */}
- {view === 'register' && ( -
- - setName(e.target.value)} - className="w-full px-3 py-2 text-sm rounded border border-warm-200 dark:border-warm-700 bg-white dark:bg-warm-900 text-navy-950 dark:text-white placeholder-warm-400 dark:placeholder-warm-500 outline-none focus:ring-2 ring-teal-400 dark:ring-teal-500" - placeholder="Your name (optional)" - /> -
- )} -