Skip to content

Feature/cleanup and emails#505

Merged
idormenco merged 4 commits intomainfrom
feature/cleanup-and-emails
May 28, 2024
Merged

Feature/cleanup and emails#505
idormenco merged 4 commits intomainfrom
feature/cleanup-and-emails

Conversation

@idormenco
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented May 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
votemonitor ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2024 10:02am

var template = EmailTemplateLoader.GetTemplate(EmailTemplateType.InvitationExistingUser);

var body = template
.Replace("~$name$~", props.FullName)

Check warning

Code scanning / CodeQL

Dereferenced variable may be null

Variable [props](1) may be null at this access because of [this](2) potential null argument.
var template = EmailTemplateLoader.GetTemplate(EmailTemplateType.InvitationNewUser);

var body = template
.Replace("~$name$~", props.FullName)

Check warning

Code scanning / CodeQL

Dereferenced variable may be null

Variable [props](1) may be null at this access because of [this](2) potential null argument.
var body = template
.Replace("~$cdnUrl$~", props.CdnUrl)
.Replace("~$resetPasswordUrl$~", props.ResetPasswordUrl);
.Replace("~$name$~", props.FullName)

Check warning

Code scanning / CodeQL

Dereferenced variable may be null

Variable [props](1) may be null at this access because of [this](2) potential null argument.
Section,
Text,
} from "@react-email/components";
import { Heading, Link, Text } from "@react-email/components";

Check notice

Code scanning / CodeQL

Unused variable, import, function or class

Unused import Heading.
@idormenco idormenco merged commit 9710e66 into main May 28, 2024
@idormenco idormenco deleted the feature/cleanup-and-emails branch May 28, 2024 10:08
idormenco added a commit that referenced this pull request Oct 4, 2024
* Merge remote-tracking branch 'origin/feat/emails'

* added logo

* fix build

* Cleanup
idormenco added a commit that referenced this pull request Oct 4, 2024
* Merge remote-tracking branch 'origin/feat/emails'

* added logo

* fix build

* Cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant