Dynamically generate event calendar #332

Merged
a3thakra merged 25 commits from adi-ical into main 1 year ago
Collaborator

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 1 year ago
a3thakra requested review from n3parikh 1 year ago
a3thakra added 1 commit 1 year ago
53f5ca675d Undo time change for events
Poster
Collaborator

I'll revert #53f5ca before merging

I'll revert #53f5ca before merging
n3parikh approved these changes 1 year 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
Collaborator

same date and time, yes.

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

Reviewers

n3parikh approved these changes 1 year 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

No dependencies set.

Reference: www/www-new#332
Loading…
There is no content yet.