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.

Sign up for GitHub

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

Jump to bottom

Add links to API Reference for Common Error module #2064

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add links to API Reference for Common Error module #2064

wants to merge 1 commit into from

Conversation

sgchoi5
Copy link
Contributor

@sgchoi5 sgchoi5 commented Feb 24, 2025

Added links to API Reference for guide search feature

@TizenDocsBot
Copy link
Collaborator

Can one of the admins verify this patch?

@TizenDocsBot
Copy link
Collaborator

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

@sgchoi5
Copy link
Contributor Author

sgchoi5 commented Feb 24, 2025 •
edited
Loading

Unable to access: https://docs1.stg.tizen.org/staging/2064
https://docs1.stg.tizen.org/staging/2064/application/native/guides/error/error-handling/

Both page just display

403 Forbidden

@sgchoi5 sgchoi5 force-pushed the add_link_to_api_reference_for_common_error branch from 2b80d75 to 4f5f54d Compare February 25, 2025 02:07
@TizenDocsBot
Copy link
Collaborator

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

@sgchoi5 sgchoi5 force-pushed the add_link_to_api_reference_for_common_error branch from 4f5f54d to 4590de2 Compare February 27, 2025 01:10
@TizenDocsBot
Copy link
Collaborator

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

safir-srbd
safir-srbd reviewed Feb 27, 2025
View reviewed changes
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.

I have gone through the changes that you have made. I have 1 comment, please check.

docs/application/native/guides/error/error-handling.md Outdated
Comment on lines 31 to 35
```
Remarks:
The specific error code can be obtained using the get_last_result() function. Error codes are described in Exception section.
The specific error code can be obtained using the get_last_result() function in [Common Error](../../api/common/latest/group__CAPI__COMMON__ERROR.html). Error codes are described in Exception section.
To see all of the available error values, please refer to the Exceptions section in each API function description.
For example:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since the " ``` " symbol is involved, I cannot properly provide code suggestions.

Please move the " ``` " in Line 31 to Line 35, just before the "For example:"

Otherwise it appears like this.

image

@sgchoi5 sgchoi5 force-pushed the add_link_to_api_reference_for_common_error branch from 4590de2 to 944f20d Compare February 27, 2025 06:05
@sgchoi5
Copy link
Contributor Author

sgchoi5 commented Feb 27, 2025

Moved the link to other place outside of code example area. but, not yet checked using staging server (it is still 403 forbidden)

@TizenDocsBot
Copy link
Collaborator

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

@sgchoi5 sgchoi5 force-pushed the add_link_to_api_reference_for_common_error branch from 944f20d to c35eca8 Compare February 27, 2025 06:32
@TizenDocsBot
Copy link
Collaborator

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

@sgchoi5 sgchoi5 force-pushed the add_link_to_api_reference_for_common_error branch from c35eca8 to 3c6a0ca Compare February 27, 2025 06:53
@TizenDocsBot
Copy link
Collaborator

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

@pkosko pkosko mentioned this pull request Feb 27, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@safir-srbd safir-srbd safir-srbd left review comments

@asm-foysal asm-foysal Awaiting requested review from asm-foysal asm-foysal is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sgchoi5 @TizenDocsBot @safir-srbd