-
Notifications
You must be signed in to change notification settings - Fork 453
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add MediaVision explanation in overview.md of what-is-tizen #1615
Conversation
Signed-off-by: Tae-Young Chung <[email protected]>
checkout the webpage https://docs1.stg.tizen.org/staging/1615 |
Check Broken Link: Not found any broken link |
🔑 [Keyword] docs/platform/what-is-tizen/overview.md
Need help? YAML meta tag guide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tyoungroy Hello,
I have reviewed only the updated part of the file on this PR and suggested some changes.
Please look into them.
@asm-foysal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @tyoungroy,
Hope you are doing well. Since this PR has been labeled urgent, please incorporate the changes at your earliest convenience. We are hoping to merge this PR as soon as possible. Thanks.
Signed-off-by: Tae-Young Chung <[email protected]>
checkout the webpage https://docs1.stg.tizen.org/staging/1615 |
Check Broken Link: Not found any broken link |
🔑 [Keyword] docs/platform/what-is-tizen/overview.md
Need help? YAML meta tag guide |
checkout the webpage https://docs1.stg.tizen.org/staging/1615 |
Check Broken Link: Not found any broken link |
🔑 [Keyword] docs/platform/what-is-tizen/overview.md
Need help? YAML meta tag guide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tyoungroy, thanks for incorporating the changes on a timely manner.
@asm-foysal @mijin85cho I am approving this PR.
Please proceed with merging of this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tyoungroy, there are a few changes to address.
Please check the Github suggestions that I have made to find the changes and implement them at your earliest convenience.
Signed-off-by: Tae-Young Chung <[email protected]>
checkout the webpage https://docs1.stg.tizen.org/staging/1615 |
Check Broken Link: Not found any broken link |
🔑 [Keyword] docs/platform/what-is-tizen/overview.md
Need help? YAML meta tag guide |
Sorry for the interrupt with other issue : Hello, @tyoungroy BTW, Would you please check below issue also? An issue is reported under native/guides/multimedia/face-detection.md, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tyoungroy, thanks for incorporating all the suggested changes.
@asm-foysal @mijin85cho I am approving this PR.
Please proceed with merging this PR.
Signed-off-by: Tae-Young Chung [email protected]