Skip to content

Comments

[WIP] HubSpot: Add UpdatedCompany trigger#632

Closed
Copilot wants to merge 4 commits intodevfrom
copilot/fix-631
Closed

[WIP] HubSpot: Add UpdatedCompany trigger#632
Copilot wants to merge 4 commits intodevfrom
copilot/fix-631

Conversation

Copy link
Contributor

Copilot AI commented Jul 14, 2025

  • Add WATCHED_PROPERTIES_COMPANY to commons.js for company properties tracking
  • Create GetCompaniesProperties component for property metadata and transformations
  • Create UpdatedCompany trigger component following same pattern as UpdatedContact/UpdatedDeal
  • Create tests for UpdatedCompany component
  • Test the implementation and ensure linting passes

This implements the HubSpot UpdatedCompany trigger component following the same code style and patterns as other HubSpot components.

Fixes #631.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: jirihofman <12988096+jirihofman@users.noreply.github.com>
Copy link
Contributor

@jirihofman jirihofman left a comment

Choose a reason for hiding this comment

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

Retry. The last run didn't finish, see #632

I changed the firewall settings now.

Copy link
Contributor Author

Copilot AI commented Jul 14, 2025

Retry. The last run didn't finish, see #632

I changed the firewall settings now.

I'll continue with the implementation. I can see there were some formatting issues in the previous attempt that I'll fix first, then complete the remaining components.

Copy link
Contributor

@jirihofman jirihofman left a comment

Choose a reason for hiding this comment

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

It did not finish again: #632

Please please try again.

Co-authored-by: jirihofman <12988096+jirihofman@users.noreply.github.com>
Copy link
Contributor

@jirihofman jirihofman left a comment

Choose a reason for hiding this comment

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

The session didn't finish #632 @copilot

Try again.

@jirihofman
Copy link
Contributor

@copilot try again

Copy link
Contributor Author

Copilot AI commented Jul 17, 2025

The session didn't finish #632 @copilot

Try again.

I'll continue with the implementation now. The commons.js file already has the necessary company properties and helper functions. I'll create the missing GetCompaniesProperties and UpdatedCompany components.

@copilot try again

Continuing with the HubSpot UpdatedCompany implementation. I can see the commons.js file has been updated with company properties support. Now I'll create the remaining components following the same pattern as UpdatedContact and UpdatedDeal.

@jirihofman
Copy link
Contributor

Retry again @copilot

Copy link
Contributor

@jirihofman jirihofman left a comment

Choose a reason for hiding this comment

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

Retry again @copilot

@jirihofman
Copy link
Contributor

@copilot Be so kind and try again

Co-authored-by: jirihofman <12988096+jirihofman@users.noreply.github.com>
@jirihofman
Copy link
Contributor

@copilot try again

@jirihofman
Copy link
Contributor

Dear @copilot would you be so kind and try again?

@jirihofman jirihofman closed this Jan 21, 2026
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.

HubSpot: Add UpdatedCompany trigger

2 participants