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

Is SendGridMessage.OpenTracking.SubstitutionTag Optional? #638

Closed
kspearrin opened this issue Nov 10, 2017 · 3 comments
Closed

Is SendGridMessage.OpenTracking.SubstitutionTag Optional? #638

kspearrin opened this issue Nov 10, 2017 · 3 comments
Labels
difficulty: easy fix is easy in difficulty status: help wanted requesting help from the community type: community enhancement feature request not on Twilio's roadmap

Comments

@kspearrin
Copy link

SendGridMessage has an OpenTracking method that requires a enable boolean and substitutionTag string:

public void SetOpenTracking(bool enable, string substitutionTag);

I don't plan to use the substitutionTag. Should I just pass null?

Ex:

sendGridMessage.SetOpenTracking(true, null);

Where will the tracking pixel be placed in this case? At the very bottom of the email?

@thinkingserious thinkingserious added status: help wanted requesting help from the community type: question question directed at the library labels Nov 10, 2017
@thinkingserious
Copy link
Contributor

Hello @kspearrin,

Yes, the substituionTag is optional and we should update this function to reflect that. I'm going to leave this issue open for that purpose.

With regards to the tracking pixel, I'm not sure where it gets placed. I would recommend sending a test email and viewing the source. Alternatively, you can reach out to our support team.

With Best Regards,

Elmer

@thinkingserious thinkingserious added type: community enhancement feature request not on Twilio's roadmap difficulty: easy fix is easy in difficulty up-for-grabs and removed type: question question directed at the library labels Mar 1, 2018
KoditkarVedant pushed a commit to KoditkarVedant/sendgrid-csharp that referenced this issue Jun 16, 2018
KoditkarVedant added a commit to KoditkarVedant/sendgrid-csharp that referenced this issue Jun 16, 2018
thinkingserious added a commit that referenced this issue Aug 21, 2018
…lue-for-substitutiontag

#638  Add default value for SubstitutionTag parameter
@Niladri24dutta
Copy link
Contributor

@thinkingserious is this still open? Can i update the description of the function to add the optional part?

@thinkingserious
Copy link
Contributor

Since there has been no activity on this issue since March 1, 2020, we are closing this issue. Please feel free to reopen or create a new issue if you still require assistance. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: easy fix is easy in difficulty status: help wanted requesting help from the community type: community enhancement feature request not on Twilio's roadmap
Projects
None yet
Development

No branches or pull requests

4 participants