Skip to content

Fix team member yaml files LinkedIn/Twitter ('p', 'r')

Daniel Schömer requested to merge (removed):13191-team-member-data-yaml-p-r into master

Why is this change being made?

For some GitLab Inc. team members, the LinkedIn and/or Twitter URL is rendered incorrectly at https://about.gitlab.com/company/team/.

(Copied from #13191 (closed)) For example, the LinkedIn URL of Ben Cowdry is rendered as https://www.linkedin.com/in/https://www.linkedin.com/in/bencowdry instead of https://www.linkedin.com/in/bencowdry. The "data yaml" of Ben Cowdry contains the full LinkedIn URL in property linkedin.

image

This MR updates related yaml files starting with letters 'p' and 'r', located at data/team_members/person/p and data/team_members/person/r. It reduces property linkedin to contain the LinkedIn URL fragment after /in/, and property twitter to contain the Twitter handle.

Author Checklist

  • Provided a concise title for the MR
  • Added a description to this MR explaining the reasons for the proposed change, per say-why-not-just-what
    • Copy/paste the Slack conversation to document it for later, or upload screenshots. Verify that no confidential data is added.
  • Assign reviewers for this change to the correct DRI(s)
    • If the DRI for the page/s being updated isn’t immediately clear, then assign it to one of the people listed in the "Maintained by" section in on the page being edited.
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in #mr-buddies.
  • If the changes affect team members, or warrant an announcement in another way, please consider posting an update in #whats-happening-at-gitlab linking to this MR.
    • If this is a change that directly impacts the majority of global team members, it should be a candidate for #company-fyi. Please work with internal communications and check the handbook for examples.

Part of a series of MRs to close #13191 (closed)

Merge request reports