Skip to content
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

Update mc action destination field labels and descriptions.ts #2173

Closed
wants to merge 2 commits into from

Conversation

shannoncawley
Copy link

Updated the labels of some of the fields for this destination along with their descriptions to better align with the sendgrid docs and field names within sendgrid.

Updated the labels of some of the fields for this destination along with their descriptions to better align with the sendgrid docs and field names within sendgrid. The following fields were updated: Line ID, Facebook ID, WhatsApp, Unique Name, and Phone Number ID.

Testing

Include any additional information about the testing you have completed to
ensure your changes behave as expected. For a speedy review, please check
any of the tasks you completed below during your testing.

  • Added unit tests for new functionality
  • Tested end-to-end using the local server
  • [Segmenters] Tested in the staging environment

Updated the labels of some of the fields for this destination along with their descriptions to better align with the sendgrid docs and field names within sendgrid.
removed asterisks that were originally added
@tcgilbert tcgilbert self-assigned this Jul 12, 2024
@tcgilbert
Copy link
Contributor

closing this and opening new one with types here: #2179

@tcgilbert tcgilbert closed this Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants