AdobeDocs / aep-mobile-sdkdocs

Documentation for AEP Mobile SDKs
Apache License 2.0
6 stars 36 forks source link

[Places] - Review Feedback #110

Open PravinPK opened 1 year ago

PravinPK commented 1 year ago

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 Need space below heading "Configure the Places extension in Data Collection UI"
Screen Shot 2022-10-26 at 2 13 48 PM
  1. Places - Overiew Page Need space below heading "Download and import the Places extension"
Screen Shot 2022-10-26 at 2 14 59 PM
  1. 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/)

  2. Places - Overiew Page Under section : Register the Places extension with Mobile Core In tabs iOS (AEP 3.x) and iOS (ACP 2.x) Rename "yourLaunchEnvironmentID" to "yourAppId"

  3. Places - Overiew Page Under section Additional Location Service Resource

Remove the links to Places monitor

  1. Make new section - Release notes and migrate the release notes from Experience league docs
calebk1 commented 1 year ago

@PravinPK - styling issues (such as the lack of a gap between the header, lack of a gap between the tab block and header, and the extra space in the tab) are issues we (as in the docs team) can't resolve.

Please create tickets in the DEVSITE Jira to get these addressed!

The rest of these are issues I can resolve, so I'll try to do this as soon as possible!

emdobrin commented 1 year ago

@calebk1 can you double check which of these items are still open and help with resolving them?

calebk1 commented 1 year ago

Ack'd - will do some time next week most likely.