Skip to content

Conversation

@seanspeaks
Copy link
Contributor

@seanspeaks seanspeaks commented Mar 21, 2025

TL;DR

Removed unused FormData import from the Frontify API module.

What changed?

Removed the import statement for FormData from the node:form-data module in the Frontify API client file, as it was not being used in the code.

How to test?

  1. Run the existing tests for the Frontify API client to ensure functionality is not affected
  2. Verify that the application builds and runs correctly without the removed import

Why make this change?

This change improves code cleanliness by removing an unused dependency import. Removing unused imports helps reduce potential confusion during code maintenance and slightly improves module load time.

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @friggframework/api-module-frontify@1.3.1-canary.31.d76d004.0
# or 
yarn add @friggframework/api-module-frontify@1.3.1-canary.31.d76d004.0

Version

Published prerelease version: @friggframework/api-module-frontify@2.0.0-next.6

Changelog

🐛 Bug Fix

  • @friggframework/api-module-frontify
    • node:form-data doesn't exist. Thanks Cursor/AI and my like of node knowledge. #31 (@seanspeaks)
  • @friggframework/api-module-microsoft-teams, @friggframework/api-module-slack, @friggframework/api-module-42matters, @friggframework/api-module-asana, @friggframework/api-module-attio, @friggframework/api-module-connectwise, @friggframework/api-module-contentful, @friggframework/api-module-contentstack, @friggframework/api-module-crossbeam, @friggframework/api-module-deel, @friggframework/api-module-frontify, @friggframework/api-module-google-calendar, @friggframework/api-module-google-drive, @friggframework/api-module-helpscout, @friggframework/api-module-hubspot, @friggframework/api-module-ironclad, @friggframework/api-module-linear, @friggframework/api-module-salesforce, @friggframework/api-module-stripe, @friggframework/api-module-unbabel-projects, @friggframework/api-module-unbabel, @friggframework/api-module-zoho-crm, @friggframework/api-module-zoom

Authors: 1

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@seanspeaks seanspeaks marked this pull request as ready for review March 21, 2025 19:54
@seanspeaks seanspeaks merged commit 665c057 into next Mar 21, 2025
4 checks passed
Copy link
Contributor Author

Merge activity

  • Mar 21, 3:54 PM EDT: A user merged this pull request with Graphite.

@seanspeaks
Copy link
Contributor Author

🚀 PR was released in @friggframework/api-module-frontify@2.0.0-next.6 🚀

@seanspeaks seanspeaks added the prerelease This change is available in a prerelease. label Mar 21, 2025
@seanspeaks seanspeaks deleted the frontify-node-modules branch August 26, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

prerelease This change is available in a prerelease. release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants