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

Create profile-properties.md #794

Merged
merged 3 commits into from
Sep 19, 2023
Merged

Create profile-properties.md #794

merged 3 commits into from
Sep 19, 2023

Conversation

chrisseldo
Copy link
Contributor

Adding a separate profile props page to outline specs on the new data type

Amplitude Developer Docs PR

Description

Adding a separate profile props page to outline specs on the new data type

Deadline

ASAP :)

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

Adding a separate profile props page to outline specs on the new data type
@github-actions
Copy link

github-actions bot commented Jun 9, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
⚠️ MARKDOWN markdownlint 1 16 0.34s

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

@chrisseldo
Copy link
Contributor Author

@bgiori @justin-fiedler - I'm not sure I understand why this is failing. Would you be able to help?

@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-794.d19s7xzcva2mw3.amplifyapp.com

@kevinpagtakhan
Copy link
Contributor

@justin-fiedler @bgiori can we get this reviewed?

Copy link
Contributor

@markzegarelli markzegarelli left a comment

Choose a reason for hiding this comment

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

Hi @chrisseldo I'm the new Head of Dev Docs, and found this PR in the queue.

If this PR is still valid, please take a look at my edits and we'll also need to figure out where we want this to appear in the nav.

docs/data/profile-properties.md Outdated Show resolved Hide resolved
Customers can now bring in non-behavioral, customer profile data from their data warehouse to merge it with the existing behavioral product data already living in Amplitude. These values will always display the most current value being synced from the customer’s data warehouse.

## Setup
To set up a Profile Property in Amplitude, you must connect Amplitude to your data warehouse. Once connected, and you reach the data configuration section, you must select the “Warehouse Props” data type in the dropdown. From there, there are two minimum requirements for the import: a user identifier (user_id) and a profile property. Note: you may add more than one warehouse property per import, but there must be at least one per import.
Copy link
Contributor

Choose a reason for hiding this comment

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

Need some additional context here, and we should format this in an ordered list.

docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Outdated Show resolved Hide resolved
docs/data/profile-properties.md Show resolved Hide resolved
@justin-fiedler justin-fiedler removed their request for review August 31, 2023 16:17
@markzegarelli markzegarelli merged commit 267e8e3 into main Sep 19, 2023
3 of 4 checks passed
@markzegarelli markzegarelli deleted the chrisseldo-patch-2 branch September 19, 2023 18:00
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.

3 participants