Skip to content

Add redirect handler#5

Merged
cmbartschat merged 2 commits intomainfrom
christoph-b2-CS624289
Mar 17, 2026
Merged

Add redirect handler#5
cmbartschat merged 2 commits intomainfrom
christoph-b2-CS624289

Conversation

@cmbartschat
Copy link
Contributor

@cmbartschat cmbartschat commented Mar 16, 2026

Context

8thwall.com is redirecting to 8thwall.org. When there is a path specified, such as https://www.8thwall.com/tutorials, we end up on https://8thwall.org/?site_path=tutorials.

Optimally this logic would be server side, but since github pages is static, it needs to be in javascript.

Rather than using an effect, which would be mounted much later in the page load process, injecting the script tag inline in the head gives the fastest redirect logic we can muster.

This is the first of possibly many redirects we may want to add. This URL specifically is in the sidebar of the desktop app.

Testing

npm start

Same behavior on production builds with:

npm run build && npm run serve

@cmbartschat cmbartschat requested a review from kaci8 March 16, 2026 23:03
@cmbartschat cmbartschat added this pull request to the merge queue Mar 17, 2026
Merged via the queue into main with commit f9c6d3f Mar 17, 2026
1 check passed
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.

2 participants