lib-user: Refactor link urls for internal testing

[mcbouslog]

Package

• lib-user

Describe your changes

• refactor links for internal testing, replacing https://www.zooniverse.org with https://fe-root.preview.zooniverse.org
• refactor links for app-root related routes with '/' (i.e. from 'https://www.zooniverse.org/groups/12345/contributors' to '/groups/12345/contributors')

How to Review

Helpful explanations that will make your reviewer happy:

• What Zooniverse project should my reviewer use to review UX?
• What user actions should my reviewer step through to review this PR?
  • from app-root, for production resources, run yarn build then yarn start (or for staging resources run yarn dev)
  • check links (and links that look like buttons) on:
    • user stats page (/users/[login]/stats)
    • my groups page (/users/[login]/groups)
    • group stats page (/groups/[group ID])
    • group contributors page (/groups/[group ID]/contributors)
• Which storybook stories should be reviewed? n/a
• Are there plans for follow up PR’s to further fix this bug or develop this feature? see linked GitHub project

Checklist

PR Creator - Please cater the checklist to fit the review needed for your code changes.
PR Reviewer - Use the checklist during your review. Each point should be checkmarked or discussed before PR approval.

General

• Tests are passing locally and on Github
• Documentation is up to date and changelog has been updated if appropriate
• You can yarn panic && yarn bootstrap or docker-compose up --build and FEM works as expected
• FEM works in all major desktop browsers: Firefox, Chrome, Edge, Safari (Use Browserstack account as needed)
• FEM works in a mobile browser

General UX

• Can sign-in and sign-out
• The component is accessible
  • Can be used with a screen reader BBC guide to VoiceOver
  • Can be used from the keyboard WebAIM guide to keyboard testing
  • It is passing accessibility checks in the storybook

Refactoring

• The PR creator has described the reason for refactoring
• The refactored component(s) continue to work as expected

[coveralls]

coverage: 79.209% (+0.01%) from 79.195%
when pulling fdf3a12 on lib-user_links
into 48007fa on master.

[kieftrav]

One small suggestion and a question. Otherwise LGTM!

[kieftrav]

Is there a reason this one is still getting skipped?

[mcbouslog]

Great question! Eventually there'll be an All Projects page and the "see more" link will be added back and this test can be re-enabled. I've added a comment in the test code accordingly (should have done initially 🤦‍♂️ )

[coveralls]

coverage: 79.225% (+0.03%) from 79.192%
when pulling 23aa06a on lib-user_links
into cdd1c1c on master.

[coveralls]

coverage: 79.184% (-0.008%) from 79.192%
when pulling 26bd531 on lib-user_links
into cdd1c1c on master.