microsoftgraph / microsoftgraph/msgraph-sdk-python

IdentitySet class does not have 'phone' property

Open
#1,424 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

dependency:metadata type:bug
Dominant language
Python
Stars
630
Forks
96
Avg merge
15h 20m
Merged PRs (30d)
3

Description

Describe the bug

When trying to make a phone call, I specify the IdentitySet of the Target user like this:

    targets = [
        InvitationParticipantInfo(
            identity = IdentitySet(
                phone = Identity(
                    id = "+mynumber"
                )
            )
        )
    ]

However I get the following error "IdentitySet.init() got an unexpected keyword argument 'phone'" stating that this parameter is not available for the IdentitySet class.
This works when calling from the rest API

Expected behavior

Parameter should be present in the class, just like the user parameter for calling an teams userid

How to reproduce
target_identity = IdentitySet(phone=Identity(id=graph_p.phone_number))
target = InvitationParticipantInfo(
    identity = target_identity
)
request_body = Call(
    callback_uri = "myballback",
    tenant_id = tenant_id,
    source = ParticipantInfo(
        identity = IdentitySet(
            application = Identity(
                id = bot_instance_id
            )
        )
    ),
    targets = [target],
    requested_modalities = [Modality.Audio],
    media_config = ServiceHostedMediaConfig(
        pre_fetch_media = []
    )
)
SDK Version

No response

Latest version known to work for scenario above?

No response

Known Workarounds

Use IdentitySet additional_data parameter in order to inject the phone parameter to the body

Debug output
Click to expand log ```
</details>


### Configuration

_No response_

### Other information

_No response_

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the IdentitySet model entry point and reproduce the IdentitySet(phone=Identity(...)) construction from the issue. Compare the supported constructor parameters with the REST API shape and verify that a phone target can be represented in the generated request body without using additional_data.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.