Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

AMP-84095 add description of Braze User Alias Label #964

Merged
merged 7 commits into from
Sep 7, 2023

Conversation

sina-jamshidi
Copy link
Contributor

@sina-jamshidi sina-jamshidi commented Sep 7, 2023

Amplitude Developer Docs PR

Requested by NBA and RBI to be able to customize their user alias label so that it matches up with their alias cohort sync label

Description

Description added for new field in Braze event streaming

Deadline

When do these changes need to be live on the site?
By EOD when corresponding code changes are merged

Change type

  • Doc bug fix. Fixes #[insert issue number]. Amplitude contributors include Jira issue number.
  • Doc update.
  • New documentation.
  • Non-documentation related fix or update.

PR checklist:

  • My documentation follows the style guidelines of this project.
  • I previewed my documentation on a local server using mkdocs serve.
  • Running mkdocs serve didn't generate any failures.
  • I have performed a self-review of my own documentation.

@amplitude-dev-docs

@github-actions
Copy link

github-actions bot commented Sep 7, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ MARKDOWN markdownlint 3 0 0.32s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

docs/data/destinations/braze.md Outdated Show resolved Hide resolved
docs/data/destinations/braze.md Outdated Show resolved Hide resolved
@aws-amplify-us-east-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-964.d19s7xzcva2mw3.amplifyapp.com

@sina-jamshidi
Copy link
Contributor Author

Hmm looks like spell check is not failing for my changes, it's due to formatguide.md

@markzegarelli
Copy link
Contributor

@sina-jamshidi Ah, yes, sorry about that. I just added that this morning.

Let me update those so they aren't checked. If this is ready to go, (I just approved it), I can also force push to get around it.

Hmm looks like spell check is not failing for my changes, it's due to formatguide.md

@markzegarelli markzegarelli merged commit 1c0a8f3 into main Sep 7, 2023
2 of 4 checks passed
@markzegarelli markzegarelli deleted the AMP-84095-add-braze-user-alias-label branch September 7, 2023 20:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants