Fix HubSpot blog author name

[djohalo2]

Description

This pull request fixes an issue where hubspot blog author name was synced with last editor name instead of the actual linked author name. The field authorName is a bit confusing, but according to their docs indeed something else than what you would expect (https://developers.hubspot.com/docs/api-reference/latest/cms/blogs/posts/create-post#body-author-name)

To keep this change backwards compatible, we keep using the original authorName field id, but populate it with the fetched blog author name.

Changelog

• Fixed synced blog posts showing the last editor name instead of the linked author name.

Testing

• Sync a hubspot blog post and verify the author name is now populated with actual blog author that you linked to the post and not the last updated name

[Copilot]

Pull request overview

This PR corrects how HubSpot blog post authors are synced into Framer managed collections by using the linked blog author (via blogAuthorId) rather than HubSpot’s authorName value, which represents the last editor.

Changes:

• Adds an API helper to fetch a HubSpot blog author by ID.
• Enhances blog sync to request blogAuthorId (when needed), fetch author records, and map authorName to the linked author’s display name.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
plugins/hubspot/src/blog.ts	Requests blogAuthorId, fetches author names, and uses them to populate the existing authorName field.
plugins/hubspot/src/api.ts	Adds fetchBlogAuthor() and a query key for cached author lookups.

Suppressed comments (1)

plugins/hubspot/src/blog.ts:250

• syncBlogs fetches blog authors unconditionally from posts.map(post => post.blogAuthorId). If the blogAuthorId property wasn't requested/returned (e.g. when authorName isn't a field in the collection), this can produce undefined IDs and call fetchBlogAuthor(undefined), which will fail and break the sync. Gate author resolution on whether blogAuthorId was requested and filter IDs to valid non-empty strings.

    const authorIds = Array.from(new Set(posts.map(post => post.blogAuthorId)))
    const authors = authorIds.length ? await Promise.all(authorIds.map(fetchBlogAuthor)) : []
    const authorNamesById = new Map(authors.map(author => [author.id, author.displayName]))

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[huntercaron]

Silly that we have to do this at all but nice fix thank you!

[djohalo2]

@huntercaron Yeah their property name is very misleading...