fix(#288): parse automation interval strings#326
Conversation
|
@Shreya-nipunge is attempting to deploy a commit to the Dot_NotSam's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🎉 Congratulations @Shreya-nipunge! 🎉 Your pull request has been successfully merged into Paraline! Thank you so much for your valuable contribution and effort. Every single improvement helps make Paraline a better desktop experience for everyone! 🙌 🚀 What's Next?
💬 Stay Connected: Thank you again, and keep up the amazing work! 💻✨ |
Description
Fixes
themeAutomation.checkIntervalMinutessanitization so numeric strings like"15"are parsed, clamped, and stored as numbers instead of falling back to the default interval.Related Issue
Fixes #288
Changes Made
sanitizeThemeAutomation()to accept numeric strings forcheckIntervalMinutes.1..120range."abc",null,{}, and[]falling back to the default30.Testing
npm.cmd test