[SAMPLE] Review electron-app for v3.0

[sebastienlevert]

Review for v3.0 of the sample https://github.com/microsoftgraph/microsoft-graph-toolkit/tree/main/samples/electron-app

[ghost]

Hello sebastienlevert, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

[gavinbarron]

Oh boy we have some issues here:

mgt-person elements have a problem with their images:

Suspect that setting the width on the avatar-wrapper class will resolve this across the board

calls to beta endpoints are failing

Calls to beta fail with HTTP 401 and an error:

{
  "error": {
    "code": "InvalidAuthenticationToken",
    "message": "CompactToken parsing failed with error code: 80049217",
    "innerError": {
      "date": "2023-06-16T22:30:57",
      "request-id": "5572066f-d08b-49c0-8116-fc534b1c4f1d",
      "client-request-id": "360cd8f6-1c4f-0bff-5c96-e3fe400ffa98"
    }
  }
}

Quick inspection of the network calls shows that the bearer token is not being correctly set for the request and has a value of undefined in the request sent over the wire

Resolution: missing scope in provider and no incremental consent. Add the scope team.readbasic.all in main.ts

teams channel picker shows no channels when activated.

Network shows that the joinedTeams request is made and returned a valid response.

Fixed by adding the scope noted above;

[sebastienlevert]

This looks good 🤣 Good catch!