Skip to content

fix(careers): remove /careers redirect so the in-app page is reachable#5320

Merged
waleedlatif1 merged 1 commit into
stagingfrom
worktree-careers-redirect
Jul 1, 2026
Merged

fix(careers): remove /careers redirect so the in-app page is reachable#5320
waleedlatif1 merged 1 commit into
stagingfrom
worktree-careers-redirect

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • A pre-existing permanent (301) redirect /careers → https://jobs.ashbyhq.com/sim in next.config.ts shadowed the new first-party careers page shipped in feat(careers): careers page backed by the Ashby job board #5316 — visitors hitting /careers were sent straight to the Ashby board and never saw the in-app page.
  • Removed that redirect so /careers now serves the in-app careers page (which itself pulls roles from the Ashby job board).

Type of Change

  • Bug fix

Testing

Verified on the deployed site that /careers 301-redirected to jobs.ashbyhq.com/sim (<title>Sim Jobs</title>) instead of rendering the in-app page; removing the rule resolves it. Biome clean.

Note: the old rule was permanent: true (301), which browsers cache aggressively — a hard refresh / cleared cache may be needed to see the in-app page on clients that previously hit the redirect.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

A pre-existing permanent redirect /careers -> jobs.ashbyhq.com/sim shadowed the
new first-party careers page (#5316), sending visitors straight to the Ashby
board. Drop it so /careers serves the in-app page (which itself pulls the Ashby
roles).
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 1, 2026 4:43pm

Request Review

@cursor

cursor Bot commented Jul 1, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single redirect rule removal in Next config; no auth, data, or application logic changes.

Overview
Fixes /careers being shadowed by a legacy redirect so visitors get the in-app careers experience (mission hero + Ashby-backed job board) rather than an immediate hop to jobs.ashbyhq.com/sim.

The change deletes the redirects() entry for source: '/careers' with permanent: true (301) to Ashby. Other social/legacy redirects in the same block are unchanged.

Note for QA: Because the old rule was a 301, browsers may cache the external redirect until cache is cleared or a hard refresh is used.

Reviewed by Cursor Bugbot for commit 10a04c0. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR makes the in-app careers page reachable again.

  • Removes the exact /careers redirect to the Ashby-hosted jobs board.
  • Leaves the rest of the Next.js redirect and rewrite config unchanged.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/next.config.ts Removes the stale exact-path /careers redirect so the existing app route can serve the careers page.

Reviews (1): Last reviewed commit: "fix(careers): remove /careers redirect s..." | Re-trigger Greptile

@waleedlatif1 waleedlatif1 merged commit 33f9d64 into staging Jul 1, 2026
17 checks passed
@waleedlatif1 waleedlatif1 deleted the worktree-careers-redirect branch July 1, 2026 16:54
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