Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Alumni profiles are broken #1089

Closed
2 tasks done
piers-sinclair opened this issue Sep 1, 2023 · 3 comments
Closed
2 tasks done

🐛 Alumni profiles are broken #1089

piers-sinclair opened this issue Sep 1, 2023 · 3 comments
Assignees
Labels
Type: Bug A problem with existing functionality

Comments

@piers-sinclair
Copy link
Contributor

piers-sinclair commented Sep 1, 2023

Cc: @Aibono1225

Hi @bradystroud

Describe the Bug

Alumni people profiles seem to be broken

To Reproduce

Steps to reproduce the behavior:

  1. Go to https://www.ssw.com.au/rules/turn-emails-into-pbis/
  2. Click on Damian Brady
  3. see 404 error

Expected Behavior

See Damian Brady's people profile

Tasks

  • Investigate
  • Fix

Screenshots

image

Thanks!

@piers-sinclair piers-sinclair added the Type: Bug A problem with existing functionality label Sep 1, 2023
@Aibono1225
Copy link
Member

Based on my investigation, this issue occurs because in SSW.People we don't generate pages for people who no longer work for SSW.

Regarding this issue, I think we can choose one of these two solutions:

Solution 1:

Before taking users to the people profile page, check if it exists by making an HTTP request. If it's not found (404 error), don't send users to it.

Solution 2:

Leave it as it is.

@Aibono1225
Copy link
Member

Aibono1225 commented Sep 5, 2023

As per my conversation with @bradystroud , the redirection in SSW.People is not working as expected.
Normally, when people visit https://ssw.com.au/people/damian-brady, they should be redirected to https://ssw.com.au/people/damian-brady/ (with slash at the end). If this people is not working at SSW, users should then be redirected to https://ssw.com.au/people/alumni/damian-brady/.

Relevant code: https://github.com/SSWConsulting/SSW.People/blob/da001970536701628059ec5f3fdd435f17bd8b39/gatsby-node.js#L464C1-L480C6

  • Investigate
  • Make the redirect work in SSW.People

@Aibono1225
Copy link
Member

⭐ Done - The issue has been fixed.

C8980941-E131-49BB-AED4-1A25CC1DF763

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug A problem with existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants