Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Hubspot contact and company association #2947

Merged
merged 3 commits into from Feb 9, 2022

Conversation

pauloouriques
Copy link
Contributor

@pauloouriques pauloouriques commented Feb 7, 2022

Change description

This PR makes it possible to associate a contact to a company and also adds all re required tests to cover that new feature.

Checklists

Development

  • Application changes have been tested appropriately

Impact

  • Code follows company security practices and guidelines
  • Security impact of change has been considered
  • Performance impact of change has been considered
  • Possible migration needs considered (model migrations, config migrations, etc.)

Please explain any security, performance, migration, or other impacts if relevant:

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached where applicable.
  • Relevant tags have been added to the PR (bug, enhancement, internal, etc.)

@pauloouriques pauloouriques added enhancement New feature or request plugin labels Feb 7, 2022
@pauloouriques pauloouriques force-pushed the feature/hubspot-contact-company-association branch from 24ff2b8 to 80b06f4 Compare February 7, 2022 20:43
key: "companyKey",
displayName: "Company Key",
required: false,
description: "Which field in the Company uniquely represents it?",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If companyKey is optional, what will be used by default? We might want to mention the email domain will be used if this is not set.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If companyKey is not set the relationship is just ignored.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But doesn't hubspot make companies autoamtically from the email domain? Either way, 👍

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could say something like "Links a Contact to a company by something other than the Hubspot default (email domain)."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahh! Now I got what you're saying @evantahler, thank you. Brian's text will do the trick.

plugins/@grouparoo/hubspot/__tests__/.env.example Outdated Show resolved Hide resolved
key: "companyKey",
displayName: "Company Key",
required: false,
description: "Which field in the Company uniquely represents it?",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could say something like "Links a Contact to a company by something other than the Hubspot default (email domain)."

 - include interger to destination options mapping type;
 - add test companies;
 - change destination option description;
 - skip anything that starts with Company.
@pauloouriques pauloouriques merged commit 40fee16 into main Feb 9, 2022
@pauloouriques pauloouriques deleted the feature/hubspot-contact-company-association branch February 9, 2022 17:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants