microsoftgraph / microsoftgraph/msgraph-beta-sdk-python

Missing Pagination for List of Directs

Open
#612 9 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

area:documentation documentation type:documentation
Dominant language
Python
Stars
44
Forks
16
Avg merge
20h 39m
Merged PRs (30d)
3

Description

Describe the bug

I am fetching directs via client.users.by_user_id(user_id).direct_reports.get(). The list is incomplete for users with many directs. Browsing through the code a bit, it seems that this API ignores the nextLink property and only returns the first page.

Expected behavior

The list of returned directs should be complete.

How to reproduce
  • create user user_id with many directs
  • fetch direct reports of this user with client.users.by_user_id(user_id).direct_reports.get()
  • confirm not all results are returned.
SDK Version

1.11.0

Latest version known to work for scenario above?

none.

Known Workarounds

No response

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


### Configuration

- OS: Ubuntu 22.04
- arch: x86


### 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 client.users.by_user_id(user_id).direct_reports.get() and trace how the response handles the nextLink property. Reproduce the issue with a user having many direct reports, then verify that the call returns the complete list and add coverage for the multi-page case.

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
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.