Skip to content

Fix wsist.ch redirect + critical landing page polish - #39

Merged
timh8127 merged 6 commits into
mainfrom
fix/landing-redirect-and-polish
Jul 3, 2026
Merged

Fix wsist.ch redirect + critical landing page polish#39
timh8127 merged 6 commits into
mainfrom
fix/landing-redirect-and-polish

Conversation

@timh8127

@timh8127 timh8127 commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes the wsist.ch root redirect and applies five landing-page polish improvements from the task list.

  • Task 1 (critical): wsist.ch/ now issues an explicit 302 redirect to /login-page instead of a server-side rewrite. The rewrite left the browser URL at /, where the Blazor client router matched Home.razor and issued its own client-side redirect anyway — the explicit redirect removes that race.
  • Task 2: Replaced the unshipped "WSIST says" (Soon) feature card with a data-export feature card; removed the now-orphaned Landing_WsistSays/Landing_Soon resource keys and the dead .soon CSS rule.
  • Task 3: Added privacy reassurance ("we only use your name and email") to the hero note and the bottom CTA, in both English and German.
  • Task 4: Added a hello@wsist.ch contact link to the footer.
  • Task 5: Added Swiss 1–6 grade scale context to the grade-pull description for international visitors.
  • Task 6: Added a critical above-fold <style> block painting the #0c0b08 app background before any stylesheet or font loads. Preconnect hints and display=swap were already present on the Google Fonts links, so no changes were needed there.

Test plan

  • dotnet test — 35/35 passing before every commit
  • dotnet csharpier format . clean before every commit
  • Full solution builds with 0 errors
  • Verify https://wsist.ch/ 302s to https://wsist.ch/login-page after deploy

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Added an “export your data anytime” feature message on the landing page, highlighting CSV/JSON downloads.
    • Added a direct contact email link in the login page footer.
  • Bug Fixes

    • Redirects the site’s root page to the login page more reliably for landing-page visitors.
    • Updated landing-page text to clarify data usage and grading details.
  • Style

    • Applied the site background color earlier during page load for a smoother first paint.

timh8127 and others added 6 commits July 3, 2026 20:12
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…nding

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 652258b1-b234-4a1b-86f9-d3f1b84b94ff

📥 Commits

Reviewing files that changed from the base of the PR and between 56a2cde and ea20265.

📒 Files selected for processing (6)
  • WSIST/WSIST.Web/Components/App.razor
  • WSIST/WSIST.Web/Components/Pages/Login.razor
  • WSIST/WSIST.Web/Program.cs
  • WSIST/WSIST.Web/Resources/SharedResource.de.resx
  • WSIST/WSIST.Web/Resources/SharedResource.resx
  • WSIST/WSIST.Web/wwwroot/landing.css
💤 Files with no reviewable changes (1)
  • WSIST/WSIST.Web/wwwroot/landing.css

📝 Walkthrough

Walkthrough

This PR updates the landing page: the root path for wsist.ch now redirects clients to /login-page instead of rewriting internally, adds critical inline CSS for background color, replaces a "coming soon" feature with a data-export feature (updating EN/DE resource strings and removing the .soon CSS class), and adds a footer contact mailto link.

Changes

Landing page updates

Layer / File(s) Summary
Root path redirect and critical CSS
WSIST/WSIST.Web/Program.cs, WSIST/WSIST.Web/Components/App.razor
Root path / for wsist.ch/www.wsist.ch now issues an explicit client redirect to https://wsist.ch/login-page instead of an internal rewrite; an inline critical CSS block sets background color before other resources load.
Feature 6 content and footer contact link
WSIST/WSIST.Web/Components/Pages/Login.razor
The 6th feature heading switches from Landing_WsistSays to Landing_Feat6Title, and a mailto:hello@wsist.ch link is added in the footer.
Localized landing string updates (EN/DE)
WSIST/WSIST.Web/Resources/SharedResource.resx, WSIST/WSIST.Web/Resources/SharedResource.de.resx
Hero note and CTA text add privacy clarification (only name/email used), grade-pull description adds Swiss 1–6 scale context, and Feature 6 text changes from "coming soon" briefing/Android app to data export (CSV/JSON), with obsolete "soon" entries removed.
Removal of unused .soon CSS class
WSIST/WSIST.Web/wwwroot/landing.css
The .landing .soon style block is deleted since the corresponding badge content was removed.

Estimated code review effort: 2 (Simple) | ~12 minutes

Possibly related PRs

  • timh8127/WSIST#32: Prior localization work reworking Login and landing strings via IStringLocalizer, which this PR builds upon.
  • timh8127/WSIST#34: Both modify the Login.razor footer links section.
  • timh8127/WSIST#37: Both modify the wsist.ch marketing middleware in Program.cs.

Poem
A rabbit hops to wsist.ch's door,
Redirects now, not rewrites of yore.
"Coming soon" gives way to export true,
CSV and JSON, ready for you.
A dash of dark background, a mailto line —
Small hops of polish, all landing-page fine. 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the main changes: the wsist.ch redirect fix and landing page polish updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/landing-redirect-and-polish

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@timh8127
timh8127 merged commit 46b34fe into main Jul 3, 2026
2 checks passed
@timh8127
timh8127 deleted the fix/landing-redirect-and-polish branch July 28, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant