Skip to content

Conversation

tcgilbert
Copy link
Contributor

Proposed changes

Updated bucket copy per partner request.

There is one part of the docs that they wanted to update, that is not included in the index.md file.

The description in the settings display:
Screen Shot 2022-03-17 at 4 59 08 PM

They want it updated from:

  • string. The API key is your App ID. Find it under Settings (cog wheel) when logged in.

To

  • string. The API key is your app’s Tracking Key. Locate it under Settings when signed in.

This is now mapped correctly in the partner portal, so I am guessing this will be included in the next build?

Merge timing

asap

@tcgilbert tcgilbert requested a review from a team as a code owner March 17, 2022 21:04
@tcgilbert tcgilbert requested review from rchinn1 and removed request for a team March 17, 2022 21:04
tcgilbert and others added 2 commits March 17, 2022 17:06
removing auto formatting
@rchinn1 rchinn1 added the fixed-content fix to content that has outdated or wrong information label Mar 17, 2022
Copy link
Contributor

@rchinn1 rchinn1 left a comment

Choose a reason for hiding this comment

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

Thanks for adding this @tcgilbert! The Partner Portal content is mapped correctly, but I noticed “Settings” is missing the “s”. This needs to be corrected in the portal, and then we can include this update in our next docs release.
2615

@tcgilbert
Copy link
Contributor Author

Nice catch @rchinn-segment! Just updated partner portal

@rchinn1 rchinn1 merged commit c532886 into develop Mar 18, 2022
@rchinn1 rchinn1 deleted the thomas/bucket-updates branch March 18, 2022 18:01
@github-actions
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-content fix to content that has outdated or wrong information
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants