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

git_calendar/yaml2ics: Allow include to include from a url ICS #10

Merged
merged 4 commits into from
Jun 5, 2024

Conversation

rkdarst
Copy link
Member

@rkdarst rkdarst commented Jun 3, 2024

  • Allows to include from a remote ICS file
  • Remote YAML is harder since it requires being able to include other
    files (which I am not doing yet)

- Allows to include from a remote ICS file
- Remote YAML is harder since it requires being able to include other
  files (which I am not doing yet)
@bast
Copy link
Member

bast commented Jun 5, 2024

Sorry I saw this one after #11 and am now confused how the two PRs relate. @tpfau ?

@tpfau
Copy link
Contributor

tpfau commented Jun 5, 2024

This exacly how it was intended. I wanted my PR to be merged with Richards branch and will now merge Richards branch. 😄

@tpfau tpfau merged commit 0324346 into main Jun 5, 2024
2 checks passed
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.

3 participants