AdobeDocs / AdobeDocs/aep-mobile-sdkdocs

[Places] - Review Feedback

Open
#110 3 comments 0 reactions 1 assignee Claimed by @calebk1 View on GitHub
Dominant language
JavaScript
Stars
6
Forks
54
Avg merge
1d 18h
Merged PRs (30d)
10

Description

Places Extension Staging Link : https://developer-stage.adobe.com/client-sdks/documentation/places/
Places Extension Gitbook Link: https://aep-sdks.gitbook.io/docs/foundation-extensions/places

The content of the migration looks correct. But there are a few minor edits and alignment changes that can make the doc look better

1. Places - [Overiew Page](https://developer-stage.adobe.com/client-sdks/documentation/places/)
Need space below heading "Configure the Places extension in Data Collection UI"

Screen Shot 2022-10-26 at 2 13 48 PM

2. Places - [Overiew Page](https://developer-stage.adobe.com/client-sdks/documentation/places/)
Need space below heading "Download and import the Places extension"

Screen Shot 2022-10-26 at 2 14 59 PM

3. Places - All tabs
Would be nice if we can remove all the extra spaces below the tabs.
Screen Shot 2022-10-26 at 2 17 21 PM
tation/places/)

4. Places - [Overiew Page](https://developer-stage.adobe.com/client-sdks/documentation/places/)
Under section : [Register the Places extension with Mobile Core](https://developer-stage.adobe.com/client-sdks/documentation/places/#register-the-places-extension-with-mobile-core)
In tabs iOS (AEP 3.x) and iOS (ACP 2.x)
Rename "yourLaunchEnvironmentID" to "yourAppId"

5. Places - [Overiew Page](https://developer-stage.adobe.com/client-sdks/documentation/places/)
Under section [Additional Location Service Resource](https://developer-stage.adobe.com/client-sdks/documentation/places/#additional-location-service-resources)

Remove the links to Places monitor
- [Places Monitor SDK implementation](https://experienceleague.adobe.com/docs/places/using/places-ext-aep-sdks/places-monitor-extension/places-monitor-extension.html?lang=en)
- [Cordova Places Monitor SDK GitHub readme](https://github.com/adobe/cordova-acpplaces-monitor/blob/master/README.md)
- [React Native Places Monitor SDK GitHub readme](https://github.com/adobe/react-native-acpplaces-monitor/blob/master/README.md)
- [Flutter Places Monitor SDK GitHub readme](https://github.com/adobe/flutter_acpplaces_monitor/blob/master/README.md)

6. Make new section - Release notes and migrate the release notes from [Experience league docs](https://experienceleague.adobe.com/docs/places/using/release-notes.html?lang=en)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.