Is it a good idea to make a type that inherits EmailConfig?

[kapios010]

I'm trying to add a new email provider to the core package - Brevo (ex. Sendinblue)
[link to fork]

Brevo's API works in such a way, where the sender data is sent through a JSON object where the name and the email are separate, as seen below.

...
body: JSON.stringify({
	sender: {
		name: provider.from.name,
		email: provider.from.email,
	},
...

Now, the approach to this in the Mailgun provider (even though the API doesn't take an object), was to make the user change the provider name from Mailgun to let's say Example App. I myself don't think it's a good approach as this will, for example, change the default sign in button text from Sign in with Mailgun to Sign in with Example App, which could cause confusion - you're signing up to your app with your app.

I changed the EmailConfig type, rather made a new, inherited type, BrevoConfig (following the NodemailerConfig name) that does have the sender as an object with those 2 options:

interface BrevoConfig
  extends Omit<EmailConfig, "from" | "options" | "sendVerificationRequest"> {
  from?: {
    name?: string
    email: string
  }
  sendVerificationRequest: (params: {
    identifier: string
    url: string
    expires: Date
    provider: BrevoConfig
    token: string
    theme: Theme
    request: Request
  }) => Awaitable<void>
  options?: BrevoUserConfig
}

export type BrevoUserConfig = Omit<Partial<BrevoConfig>, "options" | "type">

I'm not sure whether the code that defines this interface is good as it is my first time using TS features such as Omit<>, but nevertheless, when I use the provider in the demo app it shows this error:

Type 'BrevoConfig' is not assignable to type 'Provider'.
  Type 'BrevoConfig' is not assignable to type 'EmailConfig & InternalProviderOptions'.
    Type 'BrevoConfig' is not assignable to type 'EmailConfig'.
      Types of property 'from' are incompatible.
        Type '{ name?: string | undefined; email: string; } | undefined' is not assignable to type 'string | undefined'.
          Type '{ name?: string | undefined; email: string; }' is not assignable to type 'string'.  ts(2322)

It runs just fine and works, but I'm not sure whether the error comes from VSCode because it is a demo app or maybe the type configuration should be reverted back to standard EmailConfig or should the EmailConfig type by default include separate fields for name and email.