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

[BUG] Person card - the organization diagram looks misleading #2872

Closed
Rafaelki opened this issue Nov 23, 2023 · 1 comment · Fixed by #2930
Closed

[BUG] Person card - the organization diagram looks misleading #2872

Rafaelki opened this issue Nov 23, 2023 · 1 comment · Fixed by #2930
Labels
bug Something isn't working State: In Review
Milestone

Comments

@Rafaelki
Copy link

Describe the bug
This is very minor, but hope you can take a look. The organization diagram shows lines that suggest "report lines" for people who report to the same person, see image below, lines marked in red are misleading

image

To Reproduce
Steps to reproduce the behavior:

  1. Go to the person card in the MGT playground
  2. Expand the card and click on "Diego Siciliani"
  3. In the "Reports to" section, click on "Show more"
  4. Note that there are lines from "Emily Braun" to "Megan Bowen" and from "Megan Bowen" to "Debra Berger" that suggest that "Megan Bowen" reports to "Emily Braun" and "Debra Berger" reports to "Megan Bowen" when actually the 3 of them report to "Diego Siciliani"

Expected behavior
There should not be lines between people who report to the same person. Ideally, all 3 should be in the same box:

image

Environment (please complete the following information):

  • OS: Windows 11
  • Browser: chrome
  • Framework: react
  • Context: SharePoint
  • Version: 3
@Rafaelki Rafaelki added bug Something isn't working Needs: Triage 🔍 labels Nov 23, 2023
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Graph Toolkit Nov 23, 2023
@sebastienlevert
Copy link
Contributor

This actually feels like a bug. Can you confirm @Mnickii?

@gavinbarron gavinbarron moved this from Needs Triage 🔍 to Proposed 💡 in Graph Toolkit Jan 3, 2024
@gavinbarron gavinbarron added this to the v4.0.0 milestone Jan 4, 2024
@gavinbarron gavinbarron moved this from Proposed 💡 to In Review 💭 in Graph Toolkit Jan 4, 2024
@github-project-automation github-project-automation bot moved this from In Review 💭 to Done ✔️ in Graph Toolkit Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working State: In Review
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants