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

Update EU instructions for adding Sendgrid data source #972

Merged
merged 4 commits into from
Sep 18, 2023

Conversation

ampcynthli
Copy link
Contributor

Amplitude Developer Docs PR

Description

  • Add updated text to ensure clarity for EU customers

Deadline

None

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 15, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ MARKDOWN markdownlint 1 0 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

docs/data/sources/sendgrid.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-972.d19s7xzcva2mw3.amplifyapp.com

ampcynthli and others added 2 commits September 15, 2023 16:12
@markzegarelli
Copy link
Contributor

@ampcynthli Yeah, list formatting is really quirky in this tool (and in Markdown).

I've updated the list so it renders as a sub-bullet, and also broke up the text block by region, for better scanability.

@ampcynthli
Copy link
Contributor Author

@markzegarelli Thank you so much for your help!

@ampcynthli ampcynthli merged commit c7c1cfd into main Sep 18, 2023
4 checks passed
@ampcynthli ampcynthli deleted the AMP-84201-updateSendgridSourceDoc branch September 18, 2023 16:36
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