TidyLinker.com
TypeScript
Pull Request Opened
PR #131 opened: chore: remove public job sharing pages
Summary
- Removes the public
/jobs/{jobPosting}share page, controller, and feature test after stakeholder pushback on data-protection and brand optics. - Drops the import + named route from
routes/web.php. No other code referencesjobs.show; scopedclient.jobs.*andcleaner.jobs.*are untouched. - Confirmed via prod tinker that no admin "view public page" surface exists on
main(those buttons live in open PR #130, which can be rebased once this lands).
Why now
Adele only just started sharing these links — there's no SEO weight to lose, and the data exposure outweighs the marketing benefit. The shareable flow shipped in #126; this rolls back the public surface while leaving the re-notify-on-update behaviour intact.
Test plan
-
php artisan testagainstCleaner/JobControllerTestandClient/JobControllerTest— 42 passing -
php artisan route:list— nojobs.showroute, scoped routes intact -
vendor/bin/pint --dirtyclean - After merge: rebase #130 to drop
publicUrlreferences from admin job show
+0
additions
-405
deletions
4
files changed