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

[Web][8.0] Add 8.0 to TOC #1931

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

pkosko
Copy link
Collaborator

@pkosko pkosko commented Feb 19, 2024

The goal of this PR is to add missing 8.0 reference in doc page:
image

@TizenDocsBot
Copy link
Collaborator

Can one of the admins verify this patch?

1 similar comment
@TizenDocsBot
Copy link
Collaborator

Can one of the admins verify this patch?

@pkosko
Copy link
Collaborator Author

pkosko commented Feb 19, 2024

Dear maintainers could you please verify if my change will affect TOC mentioned in PR description? Otherwise, how it could be achieved?

@TizenDocsBot
Copy link
Collaborator

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

Copy link
Collaborator

@safir-srbd safir-srbd left a comment

Choose a reason for hiding this comment

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

Dear maintainers could you please verify if my change will affect TOC mentioned in PR description? Otherwise, how it could be achieved?

@pkosko From initial checking, it seems like it does get added from the dropdown but the LNB does not appear. For this I think it is best if you contact Ta Duc Huy (huy.td@samsung.com), the site manager of Tizen Docs.

To see how it appears here, visit this link: https://docs1.stg.tizen.org/staging/1931/application/web/api/8.0/device_api/mobile/index.html.

@sgchoi5
Copy link
Contributor

sgchoi5 commented Feb 21, 2024

Dear maintainers could you please verify if my change will affect TOC mentioned in PR description? Otherwise, how it could be achieved?

@pkosko From initial checking, it seems like it does get added from the dropdown but the LNB does not appear. For this I think it is best if you contact Ta Duc Huy (huy.td@samsung.com), the site manager of Tizen Docs.

To see how it appears here, visit this link: https://docs1.stg.tizen.org/staging/1931/application/web/api/8.0/device_api/mobile/index.html.

I've sent an email to Ta Duc Huy yesterday. He is not charge of Tizen site anymore. He will forward my question email to proper person.

@pkosko
Copy link
Collaborator Author

pkosko commented Apr 4, 2024

Hello, any updates on this topic?

@kimcinoo
Copy link
Collaborator

kimcinoo commented Apr 5, 2024

I found one part that needs to be modified while checking this issue.
The following files have 7.0 information. Please update it.

./docs/application/web/api/8.0/device_api/mobile/index.xml
./docs/application/web/api/8.0/device_api/tv/index.xml
./docs/application/web/api/8.0/device_api/wearable/index.xml

@pkosko
Copy link
Collaborator Author

pkosko commented Apr 5, 2024

./docs/application/web/api/8.0/device_api/mobile/index.xml
./docs/application/web/api/8.0/device_api/tv/index.xml
./docs/application/web/api/8.0/device_api/wearable/index.xml

Thank you for notification about this - I've updated it.

@TizenDocsBot
Copy link
Collaborator

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

@kimcinoo
Copy link
Collaborator

kimcinoo commented Apr 9, 2024

ut this - I've updated it

We need to update our side as well. We will update with high priority. I apologize for any inconvenience.

@kimcinoo
Copy link
Collaborator

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

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.

None yet

5 participants