Skip to content

js: added fillPoly() and fillConvexPoly()#14849

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
thangktran:thangktran/feature/fillPoly_and_fillConvexPoly_js_binding
Jun 25, 2019
Merged

js: added fillPoly() and fillConvexPoly()#14849
opencv-pushbot merged 1 commit intoopencv:3.4from
thangktran:thangktran/feature/fillPoly_and_fillConvexPoly_js_binding

Conversation

@thangktran
Copy link
Copy Markdown
Contributor

@thangktran thangktran commented Jun 20, 2019 •
edited by alalek
Loading

Fixes: #14690

  • added tests
  • python ./platforms/js/build_js.py --build_test js_build successfully
  • node js_build/bin/tests.js all passed
force_builders=Docs,Custom
buildworker:Custom=linux-1,linux-2,linux-4
docker_image:Docs=docs-js
docker_image:Custom=javascript

alalek
alalek approved these changes Jun 25, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@opencv-pushbot opencv-pushbot merged commit 2a67375 into opencv:3.4 Jun 25, 2019
opencv-pushbot pushed a commit that referenced this pull request Jun 25, 2019
@alalek
Merge pull request #14849 from thangktran:thangktran/feature/fillPoly…
63e4b35 
…_and_fillConvexPoly_js_binding
@thangktran thangktran deleted the thangktran/feature/fillPoly_and_fillConvexPoly_js_binding branch June 25, 2019 15:07
@alalek alalek mentioned this pull request Jun 26, 2019
Merged
@WangYundong WangYundong mentioned this pull request Jul 31, 2022
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alalek alalek alalek approved these changes

Assignees

@alalek alalek

Labels

category: javascript (js) feature

Projects

None yet

Milestone

3.4.7

Development

Successfully merging this pull request may close these issues.

3 participants

@thangktran @alalek @opencv-pushbot