Skip to content

Update index.md Autopilot requires userId or email in identify events #5289

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 8, 2023

Conversation

lizkane222
Copy link
Contributor

Proposed changes

Autopilot requires either userId or email in Engage Identify events.

Merge timing

  • ASAP once approved

Related issues (optional)

@lizkane222 lizkane222 marked this pull request as ready for review September 1, 2023 00:20
@lizkane222 lizkane222 requested a review from a team as a code owner September 1, 2023 00:20
@lizkane222 lizkane222 requested review from forstisabella and removed request for a team September 1, 2023 00:20
@forstisabella
Copy link
Contributor

forstisabella commented Sep 5, 2023

Hi @lizkane222 - this destination has some autogenerated Engage content that builds directly after the content added in this PR:

Screenshot 2023-09-05 at 12 45 16 PM 2

I'm thinking maybe we change it to a FAQ section and add this item, or maybe wrap it up under the Appendices header? I'll do some troubleshooting to see where we could integrate this to increase readability.

@forstisabella
Copy link
Contributor

hey @lizkane222 - I think we could include the blurb under the Identify section of the page, for visibility:

80FA3573-8C54-42F6-8EAA-6D267C8BA096_1_201_a

@forstisabella
Copy link
Contributor

hi @lizkane222 - any thoughts on where I've moved this content?

@forstisabella forstisabella merged commit c611534 into develop Nov 8, 2023
@forstisabella forstisabella deleted the lizkane222-patch-10 branch November 8, 2023 16:47
Copy link
Contributor

github-actions bot commented Nov 8, 2023

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants