Skip to content

Add legacy profile route redirect#279

Open
GHX5T-SOL wants to merge 1 commit into
algora-io:mainfrom
GHX5T-SOL:fix/209-profile-route
Open

Add legacy profile route redirect#279
GHX5T-SOL wants to merge 1 commit into
algora-io:mainfrom
GHX5T-SOL:fix/209-profile-route

Conversation

@GHX5T-SOL
Copy link
Copy Markdown

Summary

  • route /profile/:handle through the same handle redirect logic used by /:handle
  • keep individual users going to /:handle/profile and organizations to /:handle/dashboard
  • add controller coverage for individual, organization, and unknown handles

Fixes #209

Validation

  • TEST_DATABASE_URL=postgresql://postgres:postgres@localhost:5432/algora_test mix test test/algora_web/controllers/user_controller_test.exs (3 tests, 0 failures)
  • mix format --check-formatted lib/algora_web/controllers/user_controller.ex lib/algora_web/router.ex test/algora_web/controllers/user_controller_test.exs
  • git diff --check HEAD~1..HEAD
  • git diff HEAD~1..HEAD | gitleaks detect --pipe --no-banner --redact (no leaks)

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.

Feature Request: Navigation to /profile/:handle

1 participant