Skip to content
This repository was archived by the owner on Dec 8, 2017. It is now read-only.

Most of the way towards a cal implementation for #11#24

Open
jflasher wants to merge 1 commit into18F:masterfrom
jflasher:cal_api
Open

Most of the way towards a cal implementation for #11#24
jflasher wants to merge 1 commit into18F:masterfrom
jflasher:cal_api

Conversation

@jflasher
Copy link

Using iCalendar package (from Midas fame) to generate ics files server side. The major outstanding issue here is that solr is not returning the results I'd expect. I search for id="some_id" and it returns many results which don't contain the id. I think I am just missing something (hopefully) obvious.

I set it up as a /v0/cal/record_id endpoint since that made the most sense to me for this particular issue. But it may make more sense to make it a query parameter based endpoint if there is a desire to build calendars with multiple events in the future.

Using icalendar node lib to generate ics files server side. The major outstanding issue here is that solr is not returning the results I'd expect. I search for id="some_id" and it returns many results which don't contain the id. I think I am just missing something (hopefully) obvious.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If someone "subscribes" to the calendar (such as putting it in Google Calendar), having a different UUID every time the endpoint is hit will result in the calendar app thinking they are different calendar entries. An alternative would be to use the id, or hash the id.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This happens when someone accesses the .ics file through webcal:// rather than https://

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants