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

Script is adding the string "Range" to the first cell of each Sheet being pulled / multiple ranges not updating #49

Closed
Jdwilcox211 opened this issue Aug 12, 2024 · 6 comments

Comments

@Jdwilcox211
Copy link

Jdwilcox211 commented Aug 12, 2024

I noticed this after I updated the Google Sheets script to version 20, and redeployed it. For each range being pulled once a successful request has happened it, changes the first cell contents of the sheet the range is being pulled from to the string "Range". pictures below

range
Destination

I pull 9 ranges from one workbook, with the last update I am noticing it will not pull each range every time leaving an error message to make sure script is set to Anyone which it is and each range will pull every now and then so I know its setup correctly. It seems like it is timing out. I am playing with the requestDelay but cant seem to find the sweet spot. wasn't sure if you may have some insight

@Jdwilcox211 Jdwilcox211 changed the title Script is adding the string "Range" to the first cell of each Sheet being pulled Script is adding the string "Range" to the first cell of each Sheet being pulled / multiple ranges not updating Aug 12, 2024
ryan-d-williams added a commit that referenced this issue Aug 12, 2024
Fixing bug with auto formula refresh
@ryan-d-williams
Copy link
Owner

@Jdwilcox211 thanks for finding this!

For your first issues (the A1 cell getting replaced with "Range"), this was a bug with the library. It was due to the new feature that the formulas are auto-refreshed without need to open the sheet. I fixed the bug and pushed a new version of the library (21) that should remove that bug (you'll need to change that cell back to it's original content though).

I'm unable to reproduce your second issue. That error message is just a general help statement in case people haven't deployed properly (it definitely sounds like you have though).

Are you able to share your config? If so I can see if I can reproduce the error. If you can't share the config because your sheet data is sensitive, you can try accessing the url that is printed to the console to see what error you are getting. The error on the MM implies that google is showing some kind of error page instead of returning the data. That error page might have some details that can help us track down the issue.

@Jdwilcox211
Copy link
Author

@Jdwilcox211
Copy link
Author

I also have edited the css file not sure if that matters but i am adding an updated config with the version 21 script and my main.css file.

config and css.zip

@ryan-d-williams
Copy link
Owner

@Jdwilcox211 thanks for sending those over. I was able to reproduce the issue.

I'm not entirely sure what is causing the issue. There seems to be something very strange happening with the socket communication when multiple instances of a module are added. I've added a fix for now but I'm going to continue to explore to see if it's something going on in the MM core.

Can you please verify it's working for you now so I can close?

@Jdwilcox211
Copy link
Author

Yup it seems to be working, thank you

@ryan-d-williams
Copy link
Owner

For future reference: second issue was caused by MagicMirrorOrg/MagicMirror#3523.

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

No branches or pull requests

2 participants