-
Notifications
You must be signed in to change notification settings - Fork 176
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
Adds Spaces and Chat badges to countImportant #479
Conversation
Thanks to all y'all for this recipe! I use it with G-Suite and have missed a few important messages from only seeing Ferdium count emails and not chat messages.
"node": "^20.9.0", | ||
"pnpm": "^8.10.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's not really a change that should be part of this PR scope, but I am fine with it.
awesome stuff! Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes in volta
block are invalid
}, | ||
"volta": { | ||
"node": "20.9.0", | ||
"pnpm": "8.10.0" | ||
"node": "^20.9.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Volta v1.1.1
Could not parse version "^20.9.0"
Please verify the intended version.
Error cause: encountered unexpected token: Caret
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh was mixing it up with engines, good point I will change this back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in #481
"node": "20.9.0", | ||
"pnpm": "8.10.0" | ||
"node": "^20.9.0", | ||
"pnpm": "^8.10.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
error: Could not parse version "^8.10.0"
Please verify the intended version.
Error details written to /home/node/.volta/log/volta-error-2023-12-19_19_42_57.805.log
Thanks to all y'all for this recipe!
I use it with G-Suite and have missed a few important messages from only seeing Ferdium count emails and not chat messages.
Pre-flight Checklist
Please ensure you've completed all of the following.
Description of Change
Makes the badge account for chat messages from DMs and Spaces.
Also, unlocks version lock. I ran everything just fine with Node 20.10.0 and pnpm 8.12.1