Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • concrexit concrexit
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 70
    • Issues 70
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 10
    • Merge requests 10
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • thaliathalia
  • concrexitconcrexit
  • Merge requests
  • !542

Always use user pk to lookup a user and/or member

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Sébastiaan Versteeg requested to merge fix/member-pk-profiles into master Sep 01, 2017
  • Overview 0
  • Commits 1
  • Changes 6

Closes #481 (closed)

Previous behaviour

Steps to reproduce:

  1. Go to the member page of cohort 2017.
  2. Click on Aniek den Teuling with member id 1559.
  3. Observe that you are redirected to Romy Stähli with user id 1559.

New behaviour

Steps to validate that it works:

  1. Go to the member page of cohort 2017.
  2. Click on Aniek den Teuling with member id 1559.
  3. Observe that you are redirected to Aniek den Teuling with user id 1560.
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/member-pk-profiles