Skip to content

Conversation

charholt
Copy link
Contributor

@charholt charholt commented Dec 6, 2021

Proposed changes

Kameleeon requested to update their docs according to a new hackmd they submitted.

Merge timing

  • ASAP once approved

@charholt charholt requested a review from a team as a code owner December 6, 2021 21:52
@charholt charholt requested review from stayseesong and removed request for a team December 6, 2021 21:52
@charholt
Copy link
Contributor Author

charholt commented Dec 6, 2021

Hi—Sorry! I was trying to resolve the error, not merge the PR.

@netlify
Copy link

netlify bot commented Dec 6, 2021

✔️ Deploy Preview for segment-docs ready!

🔨 Explore the source changes: 0c5733d

🔍 Inspect the deploy log: https://app.netlify.com/sites/segment-docs/deploys/61ae86358c4ed40007e342e6

😎 Browse the preview: https://deploy-preview-2254--segment-docs.netlify.app

Copy link
Contributor

@stayseesong stayseesong left a comment

Choose a reason for hiding this comment

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

@charholt left some questions that need to be addressed before merging. Thanks!

@stayseesong stayseesong added the fixed-content fix to content that has outdated or wrong information label Dec 7, 2021
stayseesong and others added 2 commits December 8, 2021 10:10
I've verified with the customer to remove the api name`Kameleoon.API.Visitor.code`

Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
@charholt
Copy link
Contributor Author

Hi @stayseesong this is all set. Thank you!

@stayseesong stayseesong merged commit 69a2e60 into develop Jan 4, 2022
@stayseesong stayseesong deleted the kameleoon_doc_update branch January 4, 2022 17:40
@github-actions
Copy link
Contributor

github-actions bot commented Jan 4, 2022

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