G
YouTube TikTok Games
Skip to content

Blueprint v200 python plugins - #4480

Merged
hezyin merged 5 commits into
apache:mainfrom
merico-ai:blueprint_v200_py_plugins
Feb 21, 2023
Merged

hezyin merged 5 commits into
apache:mainfrom
merico-ai:blueprint_v200_py_plugins

Conversation

@CamilleTeruel

Copy link
Copy Markdown
Contributor

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

What does this PR do?

Does this close any open issues?

Closes xx

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

Camille Teruel added 2 commits February 21, 2023 16:20
Extract `items` and `next_page_request` properties from `__iter__` to let
clients iterate over page manually.
@CamilleTeruel CamilleTeruel changed the title Blueprint v200 py plugins Blueprint v200 python plugins Feb 21, 2023
@CamilleTeruel
CamilleTeruel force-pushed the blueprint_v200_py_plugins branch from 8567b58 to 9a9d846 Compare February 21, 2023 17:00
@hezyin
hezyin marked this pull request as ready for review February 21, 2023 19:05

@hezyin hezyin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hezyin
hezyin merged commit 967823f into apache:main Feb 21, 2023
@keon94 keon94 mentioned this pull request Mar 29, 2023
2 of 3 tasks
@keon94 keon94 added the component/framework This issue or PR relates to the framework label Mar 29, 2023
@keon94 keon94 added this to the v0.17.0 milestone Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/framework This issue or PR relates to the framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants