Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add ical parsing support to pixlet #1087

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

quesurifn
Copy link

@quesurifn quesurifn commented Aug 17, 2024

Apps like Next Event and Universal iCal offload processing to a backend in the cloud. Cloud processing is expensive and consumer data privacy is a concern when using this architecture. This PR adds support to parse .ics files (iCalendar) to try and alleviate these concerns.

@quesurifn
Copy link
Author

quesurifn commented Aug 17, 2024

@matslina Since developing the Universal iCalendar App and inspiring Next Event, I'm adding official support to Pixlet for ICS parsing at your suggestion. Please let me know what I can do to continue to support this PR as needed until it's merged.

Thanks.

@quesurifn quesurifn changed the title feat: add ical support feat: add ical parsing support to pixlet Aug 17, 2024
@quesurifn
Copy link
Author

quesurifn commented Aug 17, 2024

@matslina Also, is there a way to test the specific test cases of the new module better? Currently I see only a render function testing if screens exist.

@quesurifn quesurifn closed this Aug 17, 2024
@quesurifn quesurifn reopened this Aug 17, 2024
@quesurifn
Copy link
Author

@matslina Hello, was wondering if I could get a review on this?

@quesurifn
Copy link
Author

@rohansingh

@quesurifn
Copy link
Author

@rohansingh Hello, just checking in here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant