Dynamically generate event calendar #332

Merged
a3thakra merged 25 commits from adi-ical into main 11 months ago
Owner

Closes #134

  • Dynamically generates the calendar before the export step
  • I tested it with google calendar, updates are propagating as expected (it takes a day for google calendar to get updated)

You can try adding this URL to your calendar to test things out: https://csclub.uwaterloo.ca/~a3thakra/csc/adi-ical/events.ics

Closes #134 - Dynamically generates the calendar before the export step - I tested it with google calendar, updates are propagating as expected (it takes a day for google calendar to get updated) You can try adding this URL to your calendar to test things out: https://csclub.uwaterloo.ca/~a3thakra/csc/adi-ical/events.ics
a3thakra added 2 commits 11 months ago
a3thakra requested review from n3parikh 11 months ago
a3thakra added 1 commit 11 months ago
53f5ca675d Undo time change for events
Poster
Owner

I'll revert #53f5ca before merging

I'll revert #53f5ca before merging
n3parikh approved these changes 11 months ago
n3parikh left a comment
Owner

Can you post a diff between the old iCS and the new generated one?

Can you post a `diff` between the old iCS and the new generated one?
---
Owner

This is already in main, can you rebase/merge in main?

This is already in main, can you rebase/merge in main?
scale: "GREGORIAN",
x: { "X-WR-RELCALID": "3359A191-B19E-4B53-BADC-DFC084FC51C9" },
events: events.reverse().map(({ metadata }) => ({
id: `${new Date(metadata.date)
Owner

Will this break if we have more than one event on the same date?

Will this break if we have more than one event on the same date?
Poster
Owner

same date and time, yes.

same date and time, yes.
a3thakra added 1 commit 11 months ago
f1f837fe03 Revert "Undo time change for events"
a3thakra force-pushed adi-ical from f1f837fe03 to 072b7926a6 11 months ago
Poster
Owner
@n3parikh here's the diff: https://www.jstoolset.com/diff/4a412168d660c28a
a3thakra merged commit 34fb652fb9 into main 11 months ago
a3thakra deleted branch adi-ical 11 months ago

Reviewers

n3parikh approved these changes 11 months ago
The pull request has been merged as 34fb652fb9.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.