community icon indicating copy to clipboard operation
community copied to clipboard

Add Multi-Calendar App

Open logandeal opened this issue 1 year ago • 6 comments

Description

This adds a new multi-calendar app. The app displays the next event (within the next 24 hours) on one or more Google calendars. By default, it grabs the next event from all of a user's calendars when no Google Calendar IDs are specified. Optionally, a user may specify a comma-delimited list of Google Calendar IDs that the app will use when displaying the next event. If there is no event in the next 24 hours, it simply tells the user that there are no events in the next 24 hours. When the current time is more than half past the current event's duration, the next event will be displayed.

This app also supports 12 and 24-hour time.

Copilot

copilot:all

logandeal avatar Jan 14 '24 00:01 logandeal

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

Next Steps

Hello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:

  • [ ] Sign the CLA if you haven't already
  • [ ] Ensure your build is green! Any problem will display a proposed solution to try out
  • [ ] Get a review, either by Tidbyt Bot or by a Tidbyt engineer

Manual Review Required

Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:

Test Details
App Dir All files are in a single app directory
🟡 Modules Usage of cache.star requires review
Original Author The original author matches the PR author

Previews

apps/multicalendar/multicalendar.star:

tidbyt[bot] avatar Jan 14 '24 00:01 tidbyt[bot]

I have read the CLA Document and I hereby sign the CLA

logandeal avatar Jan 14 '24 00:01 logandeal

@matslina Is there an issue with the code?

logandeal avatar Feb 03 '24 00:02 logandeal

Sample Gif Tidbyt

logandeal avatar Feb 04 '24 21:02 logandeal

@logandeal Thanks for the PR, and apologies for the delay. We've had a very busy couple of weeks, and all PRs have suffered as a consequence. Especially less trivial ones.

And I'm afraid this PR is not entirely trivial. I'll do my best to test it out over the next few days. We'll need to migrate the app to use Tidbyt's oauth2 credentials, which (sometimes) cascades into a painful review process with Google. Now that I look at it, I think we already have these scopes covered, which means it might be fairly straightforward. Stay tuned, and again, apologies for the delay.

matslina avatar Feb 08 '24 19:02 matslina

@matslina Any updates? I would really like to have this app get released so that we can all use multiple calendars!

logandeal avatar Mar 02 '24 18:03 logandeal

Keeping this open.

logandeal avatar Apr 01 '24 23:04 logandeal

We're automatically closing this issue because it hasn't had any activity in 30 days. If that seems like a mistake, please feel free to re-open. Thanks!

stale[bot] avatar May 03 '24 08:05 stale[bot]

Can we please keep this open? This would be super helpful for us. @logandeal has done a lot of the legwork. 🥳

kaffolder7 avatar May 03 '24 13:05 kaffolder7