Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(Google Sheets Node): Optimize 'Keep First Row' Feature to Reduce API Requests #7197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Panchin99
Copy link

Refactored the code to improve the efficiency of the "Keep First Row" feature in the Google Sheets interaction. Updated the code to fetch and update the first row data outside the loop, reducing the number of unnecessary API requests. This optimization ensures that the first row is updated only once, resulting in improved performance and avoiding quota exceeded errors. Resolves the issue of excessive API requests when using the "Keep First Row" option.

Github issue / Community forum post (link here to close automatically): #7193

@CLAassistant
Copy link

CLAassistant commented Sep 18, 2023

CLA assistant check
All committers have signed the CLA.

@github-actions

This comment was marked as outdated.

@Joffcom
Copy link
Member

Joffcom commented Sep 19, 2023

Hey @Panchin99,

I have created NODE-784 as the internal ticket to handle the review for this.

@Joffcom Joffcom added the in linear Issue or PR has been created in Linear for internal review label Sep 19, 2023
@netroy netroy changed the title Optimize 'Keep First Row' Feature to Reduce API Requests feat(Google Sheets Node): Optimize 'Keep First Row' Feature to Reduce API Requests Jun 20, 2024
Refactored the code to improve the efficiency of the "Keep First Row" feature in the Google Sheets interaction.
Updated the code to fetch and update the first row data outside the loop, reducing the number of unnecessary API requests.
This optimization ensures that the first row is updated only once, resulting in improved performance and avoiding quota exceeded errors.
Resolves the issue of excessive API requests when using the "Keep First Row" option.
Tested and verified the fix to ensure correct behavior in n8n workflows.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member in linear Issue or PR has been created in Linear for internal review node/improvement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants