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

fix(updateProperties): getFieldMask ignore gridProperties #674

Merged
merged 1 commit into from
Aug 31, 2024

Conversation

MathRobin
Copy link
Contributor

No description provided.

@theoephraim
Copy link
Owner

good catch! and THANK YOU for adding a test as well 🔥

I'll try to get this merged asap

@MathRobin
Copy link
Contributor Author

you welcome, do you mind if i do multiple dev in same PRs ? with distinct commit, i still have the problem with duplicate headers and I have a very simple solution to manage it

@theoephraim
Copy link
Owner

probably better to split - because the duplicate header stuff isn't quite as clear whereas this is a bugfix

@MathRobin
Copy link
Contributor Author

Hi, when you will be able to merge it please ?

@theoephraim theoephraim merged commit f874bfe into theoephraim:main Aug 31, 2024
1 check passed
junedomingo added a commit to junedomingo/node-google-spreadsheet that referenced this pull request Sep 23, 2024
* main:
  wiring up github action to dmno
  chore: release v4.1.4
  add dmno and remove/encrypt test-related secrets, swap jest for vitest
  fix(updateProperties): getFieldMask ignore gridProperties (theoephraim#674)
  chore: release v4.1.3
  fix bugs introduced by PRs
  update axios, fix new google-auth-library issue, fixes theoephraim#701 theoephraim#700
  Read headers in batch on getRows() (theoephraim#693)
  Update authentication.md
  allow CI manual reruns
  Update FUNDING.yml
  use correct type in docs

# Conflicts:
#	.gitignore
#	pnpm-lock.yaml
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.

2 participants