Skip to content

Reverse ETL 2.0 docs #6571

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 45 commits into from
Aug 13, 2024
Merged

Reverse ETL 2.0 docs #6571

merged 45 commits into from
Aug 13, 2024

Conversation

forstisabella
Copy link
Contributor

Proposed changes

Updated Reverse ETL docs ahead of a Q3 marketing push

Merge timing

Before the end of Q2'24, and after approval from PMs and some eng folks

Related issues (optional)

@forstisabella forstisabella added the content-update updates to content that are not new features, includes grammar fixes, added notes label May 20, 2024
@forstisabella forstisabella requested a review from a team as a code owner May 20, 2024 19:21
@forstisabella forstisabella requested review from cmastr and removed request for a team May 20, 2024 19:21
Copy link

netlify bot commented May 20, 2024

Deploy Preview for segment-docs ready!

Name Link
🔨 Latest commit 8830955
🔍 Latest deploy log https://app.netlify.com/sites/segment-docs/deploys/66ad4c4ddc07250008e1a3c9
😎 Deploy Preview https://deploy-preview-6571--segment-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

cmastr
cmastr previously approved these changes May 22, 2024
Co-authored-by: Casie Oxford <coxford@twilio.com>

<!--- need to test what information needs to be entered into Segment UI --->

Learn more about the Snowflake Account ID [here](https://docs.snowflake.com/en/user-guide/admin-account-identifier.html){:target="_blank"}.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Learn more about the Snowflake Account ID [here](https://docs.snowflake.com/en/user-guide/admin-account-identifier.html){:target="_blank"}.
Learn more about the Snowflake Account ID [here](https://docs.snowflake.com/en/user-guide/admin-account-identifier.html){:target="_blank"}.

Can we make this link description more accessible?

pwseg
pwseg previously approved these changes Aug 13, 2024
Copy link
Contributor

@pwseg pwseg left a comment

Choose a reason for hiding this comment

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

A handful of minor suggestions, but LGTM! Amazing work on this.

Co-authored-by: pwseg <86626706+pwseg@users.noreply.github.com>
Co-authored-by: pwseg <86626706+pwseg@users.noreply.github.com>
forstisabella and others added 2 commits August 13, 2024 12:54
Co-authored-by: pwseg <86626706+pwseg@users.noreply.github.com>
forstisabella and others added 4 commits August 13, 2024 12:56
Co-authored-by: pwseg <86626706+pwseg@users.noreply.github.com>
Co-authored-by: pwseg <86626706+pwseg@users.noreply.github.com>
@forstisabella forstisabella requested a review from pwseg August 13, 2024 17:31
@forstisabella forstisabella merged commit 8c6f7f4 into develop Aug 13, 2024
4 checks passed
@forstisabella forstisabella deleted the retl-docs-refresh branch August 13, 2024 17:52
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
content-update updates to content that are not new features, includes grammar fixes, added notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants