Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • mattermost-profile-plus mattermost-profile-plus
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Tools
  • mattermost
  • mattermost-profile-plusmattermost-profile-plus
  • Merge requests
  • !11

Remove profile fields from integrations

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Bruna Pinos requested to merge touch-profile into main Aug 20, 2021
  • Overview 2
  • Commits 16
  • Pipelines 3
  • Changes 1

Posts made by integrations and webhooks use the creator user and icon but these can be overridden in the ProfilePopover, but they still belong to the creator. Because of that, fields like Touch profile and Status from the integration owner are shown in the integration post. For that, this MR adds another check to see if it is an integration or a common user.

Old behaviour

image

MR Change

image

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: touch-profile