chore: remove Pro banner, GitHub link, download button, and MENA region#4398
chore: remove Pro banner, GitHub link, download button, and MENA region#4398brlanweb wants to merge 0 commit into
Conversation
|
@brlanweb is attempting to deploy a commit to the World Monitor Team on Vercel. A member of the Team first needs to authorize it. |
Greptile SummaryThis PR strips four UI features: the Pro banner, the GitHub link with star counter, the download-app button, and the MENA region from the region selector. The ProBanner and GitHub/download removals are clean, but the MENA removal is incomplete —
Confidence Score: 3/5The ProBanner and GitHub/download removals are safe, but the MENA region removal is incomplete and leaves mobile users in the Middle East with a broken region selector on first load. The region-resolution utilities and URL state validator were not updated alongside the UI removal, leaving a broken experience for Middle East users on mobile and invalidating shared links with ?view=mena. src/utils/user-location.ts and src/utils/urlState.ts need updating to remove all mena references and remap Middle East resolution to a valid region. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[App.init] --> B[resolveUserRegion]
B --> C{Region resolved}
C -->|mena via Middle East TZ or coords| D[state.resolvedLocation = mena]
C -->|other region| E[state.resolvedLocation = valid region]
D --> F[panelLayout.init]
E --> F
F --> G{isMobile?}
G -->|yes| H[MapContainer view = mena]
G -->|no| I[MapContainer view = global]
H --> J[map.onStateChanged fires]
J --> K[regionSelect.value = mena]
K --> L[No matching option - select shows blank]
I --> M[regionSelect in sync]
N[URL ?view=mena] --> O[urlState VIEW_VALUES includes mena]
O --> P[Parsed as valid view]
P --> Q[map.setView mena]
Q --> K
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[App.init] --> B[resolveUserRegion]
B --> C{Region resolved}
C -->|mena via Middle East TZ or coords| D[state.resolvedLocation = mena]
C -->|other region| E[state.resolvedLocation = valid region]
D --> F[panelLayout.init]
E --> F
F --> G{isMobile?}
G -->|yes| H[MapContainer view = mena]
G -->|no| I[MapContainer view = global]
H --> J[map.onStateChanged fires]
J --> K[regionSelect.value = mena]
K --> L[No matching option - select shows blank]
I --> M[regionSelect in sync]
N[URL ?view=mena] --> O[urlState VIEW_VALUES includes mena]
O --> P[Parsed as valid view]
P --> Q[map.setView mena]
Q --> K
Reviews (1): Last reviewed commit: "chore: remove Pro banner, GitHub link, d..." | Re-trigger Greptile |
Summary
Type of change
Affected areas
/api/*)Checklist
api/rss-proxy.jsallowlist (if adding feeds)npm run typecheck)Documentation Alignment Checklist
Screenshots