Migrate Domain Field to the new Link Field Type

[Bonapara]

Current Behavior

The domain field on the company object uses a "Text" field type.

Desired Behavior

We want to migrate the domain field to the newly introduced linked field type, allowing users to specify multiple values (several domains).

We should adapt the email syncing logic to account for companies having different domains attached. We should sync emails from any matching domain name (e.g., [email protected] or [email protected]).

We should keep the custom logic for the domain field, such as fetching the favicon.

[greptile-apps]

Disclaimer: This comment was automatically generated with AI. It can give some useful hints as to where to look for, but sometimes it can also give very inaccurate answers so takes this with a pinch of salt :).

1. Update the domain field metadata in src/models/company.js to use the Link field type:

// src/models/company.js
{
  name: 'domain',
  type: 'Link',
  multiple: true,
  ...
}

2. Modify the email syncing logic in src/services/emailSync.js to handle multiple domains:

// src/services/emailSync.js
const domains = company.domain; // Assuming domain is now an array
emails.forEach(email => {
  if (domains.some(domain => email.endsWith(`@${domain}`))) {
    // Sync email
  }
});

3. Update any relevant UI components to support the new Link field type for the domain field in src/components/CompanyForm.js:

// src/components/CompanyForm.js
<Field name="domain" component={LinkField} multiple />

References

[pacyL2K19]

I can work on this next @Bonapara

[Bonapara]

Sure @pacyL2K19, thanks for contributing!

[pacyL2K19]

Starting work on this @Bonapara
Please assign it to me

[Bonapara]

Done! Thanks @pacyL2K19

[Bonapara]

@ijreilly is it related to what you're working on?

[ijreilly]

vision: add settings on fields to be able to add business logic executed at save.
e.g.: "isDomain" setting to remove protocol