Skip to content

Conversation

@SilviaZeta
Copy link
Contributor

@SilviaZeta SilviaZeta commented May 20, 2022

Description

Change the parameter imageUid of type string, which is passed to Python and AI plugins in the request body, to imageUids of type string[], so multiple images can be used with plugins.

Checklist:

Put an x in the boxes that apply to this pull request (you can also fill these out after opening the pull request). If you're unsure about any of these, don't hesitate to leave a comment on this pull request!

  • I have read the gliff.ai Contribution Guide.
  • I have requested to pull a branch and not from main.
  • I have checked all commit message styles match the requested structure.
  • My code follows the style guidelines of this project.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have performed a self-review of my own code.
  • I have assigned 3 or less reviewers.
  • New and existing unit tests pass locally with my changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • My changes generate no new warnings.
  • I have made corresponding changes to the documentation.
  • New database changes have been committed.
  • If appropriate, I have bumped any version numbers.

@SilviaZeta SilviaZeta requested review from a team, ChasNelson1990 and ChrisBaidoo and removed request for a team May 20, 2022 07:53
@SilviaZeta SilviaZeta changed the title feat: Multi-image selection in Python and AI plugins feat!: Multi-image selection in Python and AI plugins May 20, 2022
@gliff-ai-robot gliff-ai-robot added enhancement [Improvement] Enhancement request. and removed enhancement [Improvement] Enhancement request. labels May 20, 2022
@SilviaZeta SilviaZeta marked this pull request as draft May 24, 2022 15:11
@SilviaZeta
Copy link
Contributor Author

Merged into addZoo.

@SilviaZeta SilviaZeta closed this Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants