Nancy.zhu/update mobile app #29264
Open
Nancy.zhu/update mobile app #29264
+665
−442
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
breaking up index into sections and updating images
create a page just for push notification
creating separate page for iOS accessibility features
creating separate page for widgets
change file name
change file name
updating the screenshots for mobile app docs
update with new screenshots
Preview links (active after the
|
|
Created an editorial review card for an in-depth review. |
|
Thanks a ton for the docs update! Just some feedback on the screenshots:
|
|
|
||
| ## Shortcuts and Siri suggestions | ||
|
|
||
| **Android**: Create shortcut icons for your dashboards by touching and holding the Datadog app icon, then lift your finger. If the app has shortcuts, it displays a list. Touch and hold the desired shortcut, then drag and drop it to another location on your screen to create a unique shortcut icon. |
There was a problem hiding this comment.
Suggestion: Instead of touching and holding the Datadog app icon, then lift your finger you can just use long pressing the Datadog app icon which is a more common terminology.
| @@ -0,0 +1,65 @@ | |||
| --- | |||
| title: iOS Accessibility | |||
There was a problem hiding this comment.
Title says iOS but we also have Android suggestions here.
Datadog Summary✅ Code Quality ✅ Code Security ✅ Dependencies ❌ Pipelines Next StepsThe following jobs failed due to platform errors: DataDog/documentation / sourcemaps_previewNot enough memory while persisting files during the build process. Code Owners Approval Check / check-code-owners-approvalMissing approvals from: @DataDog/documentation Code Owners Approval Check / check-code-owners-approvalMissing approvals from: @DataDog/documentation Was this helpful? Give us feedback! |
and small text updates
inputting link
updates for screenshots
updates to screenshots for ios and android + updates to links
updated widget screenshots
updates to service and dashboard screenshots
upload android critical notification screenshots
updates to the android push notification screenshots and steps
update on text
updating tabs
screenshot
fixing screenshot names
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
This PR is to update the mobile app public docs with new screenshots and separate out the sections of the doc into various pages so that they are more readable and easy to access.
Changes include:
Merge instructions
Merge readiness:
For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the
<name>/<description>convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.If your branch doesn't follow this format, rename it or create a new branch and PR.
To have your PR automatically merged after it receives the required reviews, add the following PR comment:
Additional notes