Skip to content

iso8 pr to add helpher method to use paging links in Result#9337

Closed
Hugo-Inmanta wants to merge 2 commits intoiso8from
issue/8801-add-helper-method-for-using-paging-links-iso8
Closed

iso8 pr to add helpher method to use paging links in Result#9337
Hugo-Inmanta wants to merge 2 commits intoiso8from
issue/8801-add-helper-method-for-using-paging-links-iso8

Conversation

@Hugo-Inmanta
Copy link
Copy Markdown
Contributor

follow up iso8 PR to #9306

Add Result.all() helper method.

closes #8801

Follow-up ticket for api improvement here, I tried for a while to get the plumbing right, but I couldn't.

Strike through any lines that are not applicable (~~line~~) then check the box

  • Attached issue to pull request
  • Changelog entry
  • Type annotations are present
  • Code is clear and sufficiently documented
  • No (preventable) type errors (check using make mypy or make mypy-diff)
  • Sufficient test cases (reproduces the bug/tests the requested feature)
  • Correct, in line with design
  • End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
  • If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Description

  • Short description here *

closes Add ticket reference here

Self Check:

Strike through any lines that are not applicable (~~line~~) then check the box

  • Attached issue to pull request
  • Changelog entry
  • Type annotations are present
  • Code is clear and sufficiently documented
  • No (preventable) type errors (check using make mypy or make mypy-diff)
  • Sufficient test cases (reproduces the bug/tests the requested feature)
  • Correct, in line with design
  • End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
  • If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

…urned by v2 Rest API endpoints that support paging. See [documentation](helper_method_for_paging) for more information.

 (Issue #8801, PR #9306)

Add `Result.all()` helper method.

closes #8801

Follow-up ticket for api improvement [here](#9320), I tried for a while to get the plumbing right, but I couldn't.

Strike through any lines that are not applicable (`~~line~~`) then check the box

- [ ] Attached issue to pull request
- [ ] Changelog entry
- [ ] Type annotations are present
- [ ] Code is clear and sufficiently documented
- [ ] No (preventable) type errors (check using make mypy or make mypy-diff)
- [ ] Sufficient test cases (reproduces the bug/tests the requested feature)
- [ ] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
- [ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)
@Hugo-Inmanta Hugo-Inmanta requested a review from jptrindade July 10, 2025 08:25
@Hugo-Inmanta Hugo-Inmanta added the merge-tool-ready This ticket is ready to be merged in label Jul 10, 2025
@inmantaci
Copy link
Copy Markdown
Contributor

Processing this pull request

@inmantaci
Copy link
Copy Markdown
Contributor

Merged into branches iso8 in 88cbc1f

inmantaci pushed a commit that referenced this pull request Jul 10, 2025
…urned by v2 Rest API endpoints that support paging. See [documentation](helper_method_for_paging) for more information.

 (Issue #8801, PR #9337)

follow up iso8 PR to #9306

Add `Result.all()` helper method.

closes #8801

Follow-up ticket for api improvement [here](#9320), I tried for a while to get the plumbing right, but I couldn't.

Strike through any lines that are not applicable (`~~line~~`) then check the box

- [ ] Attached issue to pull request
- [ ] Changelog entry
- [ ] Type annotations are present
- [ ] Code is clear and sufficiently documented
- [ ] No (preventable) type errors (check using make mypy or make mypy-diff)
- [ ] Sufficient test cases (reproduces the bug/tests the requested feature)
- [ ] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
- [ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)

# Description

* Short description here *

closes *Add ticket reference here*

# Self Check:

Strike through any lines that are not applicable (`~~line~~`) then check the box

- [ ] Attached issue to pull request
- [ ] Changelog entry
- [ ] Type annotations are present
- [ ] Code is clear and sufficiently documented
- [ ] No (preventable) type errors (check using make mypy or make mypy-diff)
- [ ] Sufficient test cases (reproduces the bug/tests the requested feature)
- [ ] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
- [ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)
@inmantaci inmantaci closed this Jul 10, 2025
@inmantaci inmantaci deleted the issue/8801-add-helper-method-for-using-paging-links-iso8 branch July 10, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-tool-ready This ticket is ready to be merged in

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants