Skip to content
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

Merged
merged 4 commits into from
Apr 4, 2022
Merged

Add MediaVision explanation in overview.md of what-is-tizen #1615

merged 4 commits into from
Apr 4, 2022

Conversation

tyoungroy
Copy link
Contributor

@tyoungroy tyoungroy commented Mar 28, 2022

Signed-off-by: Tae-Young Chung [email protected]

@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1615

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/platform/what-is-tizen/overview.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: Tizen, device, Samsung, market, profile, developer, Gear, tv, platform, application, connectivity, model, user, series, IoTivity, API, OCF, service, Learning, software, type, Galaxy
---

Need help? YAML meta tag guide

Copy link
Collaborator

@omar-srbd omar-srbd left a 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.

docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
@journey2w journey2w added the Urgent Review for urgent updates label Mar 30, 2022
@journey2w
Copy link
Contributor

@asm-foysal
Please review this PR after releasing Tizen Studio 4.6.

Copy link
Collaborator

@omar-srbd omar-srbd left a 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]>
@tyoungroy tyoungroy requested a review from omar-srbd April 4, 2022 05:20
@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1615

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/platform/what-is-tizen/overview.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: Tizen, device, application, developer, user, profile, Samsung, market, tv, platform, service, Gear, connectivity, series, model, API, OCF, IoTivity, software, type, ecosystem, experience
---

Need help? YAML meta tag guide

@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1615

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/platform/what-is-tizen/overview.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: Tizen, device, model, platform, inference, profile, market, service, Samsung, developer, source, Gear, feature, network, tv, application, AI, user, support, Studio, Runtime, NN
---

Need help? YAML meta tag guide

Copy link
Collaborator

@omar-srbd omar-srbd left a 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.

Copy link
Collaborator

@omar-srbd omar-srbd left a 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.

docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
docs/platform/what-is-tizen/overview.md Outdated Show resolved Hide resolved
@tyoungroy tyoungroy requested a review from omar-srbd April 4, 2022 07:13
Signed-off-by: Tae-Young Chung <[email protected]>
@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1615

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/platform/what-is-tizen/overview.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: Tizen, device, model, platform, inference, profile, market, service, Samsung, developer, source, Gear, feature, network, tv, user, application, AI, NN, Runtime, support, Studio
---

Need help? YAML meta tag guide

@mijin85cho
Copy link
Contributor

Sorry for the interrupt with other issue :

Hello, @tyoungroy

BTW, Would you please check below issue also?

#1610

An issue is reported under native/guides/multimedia/face-detection.md,
so I ask you to check.
If you're not the right person to check, would you please assign the right person to the issue?

Copy link
Collaborator

@omar-srbd omar-srbd left a 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.

@asm-foysal asm-foysal merged commit 1a5eaf3 into Samsung:master Apr 4, 2022
@omar-srbd omar-srbd mentioned this pull request Jun 14, 2022
@omar-srbd omar-srbd mentioned this pull request Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Urgent Review for urgent updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants