Comment on page

Prefer inferred types over explicit ones

In this example the constant isn't really being used anywhere else, so it doesn't add any benefit to abstract to a constant in the first place.

Instead of doing:
const teamInviteEvent: TeamInvite = {
  language: t,
  from: session.user.name,
  to: usernameOrEmail,
  teamName: team.name,
  joinLink: BASE_URL + "/settings/teams",
};
​
await sendTeamInviteEmail(teamInviteEvent);
Do:
await sendTeamInviteEmail({
  language: t,
  from: session.user.name,
  to: usernameOrEmail,
  teamName: team.name,
  joinLink: BASE_URL + "/settings/teams",
});

Prefer defaultHandler for simple API handlers

Prefer early returns

Last modified 1yr ago